Fedir Zinchuk

Results 332 comments of Fedir Zinchuk

![image](https://github.com/user-attachments/assets/d622f7c4-3fc2-4fb5-bb67-988a8d0b8a03)

I would keep `settings.json` as it is now. To avoid bloating of `package.json`. Other than that, the PR seems working, node comands still working.

Yes please. It is better to keep them apart, there mostly joomla stuff, nothing standart for `package.json` And it really huge.

I have tested this item :white_check_mark: successfully on 46da720128be4f744313c2d560ecfcba41a3afdcThis comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43781.

> or should I close it? hold on, not so fast :smile:

@dgrammatiko please reset the changes in `settings.json`, as I see it is only formating, I do not realy want it to affect the history. Thanks!

This happens because the event `onJoomlaAfterUpdate` triggered twice in UpdateModel. First time without version https://github.com/joomla/joomla-cms/blob/a7bba710c3f4447301632f67d18645ebe3ae809a/administrator/components/com_joomlaupdate/src/Model/UpdateModel.php#L885 And then again but with version https://github.com/joomla/joomla-cms/blob/a7bba710c3f4447301632f67d18645ebe3ae809a/administrator/components/com_joomlaupdate/src/Model/UpdateModel.php#L912 And the action log store log for both....

It is because `min` attribute in the nested subfrom seems buggy. Without `min="1"` it works fine. About radio switch, there already an issue #40160

Note: this also happen when nested subform have default value.

@korenevskiy Please visit https://issues.joomla.org/tracker/joomla-cms/43530 and push the button, thanks ![Screenshot 2024-05-29_23-32-36](https://github.com/joomla/joomla-cms/assets/1568198/3956e6b5-b038-468c-a2aa-e212bf358215)