Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nested Subform field with min attribute saves with error #43652

Open
niosme opened this issue Jun 14, 2024 · 2 comments
Open

Nested Subform field with min attribute saves with error #43652

niosme opened this issue Jun 14, 2024 · 2 comments

Comments

@niosme
Copy link

niosme commented Jun 14, 2024

Steps to reproduce the issue

https://www.awesomescreenshot.com/video/28634289?key=0cd544c850693dfbd16aaaccae2adae0

Subform into a subform doesnt save correctly the data. Also showon doesnt work right.

I attached a module that i used in the video and you can recreate the problem by going into the special hours tab and trying to add something there and save it. In the first basic tab everything works fine as expected.

When a subform into a subform save doesnt work as expected.

Expected result

Show on to work and save the data as we did them.

Actual result

The data and structure changes and adds more than it should

System information (as much as possible)

Joomla 5.1.0

Additional comments

mod_open_hours.zip

@Fedik
Copy link
Member

Fedik commented Jun 14, 2024

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

@Fedik Fedik changed the title Subform into a subform in xml save error Nested Subform field with min attribute saves with error Jun 14, 2024
@niosme
Copy link
Author

niosme commented Jun 14, 2024

Thanks a lot! I ll remove it then <3
Removing min attribute also fixed the show on functionallity.

If i find something it could help more i ll attach it here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants