[#38602] - [5.2] Carry through disabled attribute to sub-fields in media and subform custom fields
- Pending
- Medium
- Build: 4.2-dev
- # 38602
- Diff
- crystalenka:4.1-dev
User tests: Successful: Unsuccessful:
Pull Request for Issue #38600 .
Summary of Changes
Added some checks in the plugins and layout fields to see if the parent fields are set as disabled. If so, set subfields to disabled also.
This is IN PROGRESS, feedback welcome.
Ready for review!
Left to do: Done!
- Something funny is happening with disabled subforms that have media fields when you save them.
- Make sure buttons on repeatable subforms are disabled or hidden when subform is disabled.
Testing Instructions
- Make sure you have two users with different user groups, both with edit access to articles
- As superuser, create a custom field of type "media".
- In the "options" tab, set "display when read only" to "yes"
- In the "permissions" tab, set the edit permissions for your other user to denied.
- As superuser, create a custom field of type "subform" with at least one subfield.
- In the "options" tab, set "display when read only" to "yes"
- In the "permissions" tab, set the edit permissions for your other user to denied.
- Login as the other user and navigate to the edit form where the custom fields are visible.
Actual result BEFORE applying this Pull Request
Media and subform fields are visible AND can be edited, or appear to be editable. Rows can be added and removed. Fields can be changed and text can be entered. However, any changes will not be saved to the database.
Expected result AFTER applying this Pull Request
Media and subform fields are visible but cannot be edited. Fields for media select, alt text, etc are disabled. Rows cannot be added or removed.
Documentation Changes Required
None to my knowledge.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_fields Layout Front End Plugins |
| Labels |
Added:
?
|
||
| Category | Administration com_fields Layout Front End Plugins | ⇒ | Administration com_fields Repository NPM Change Layout Front End Plugins |
All the strange edge cases and bugs (looking at you, accessible media field...) have now been resolved to the best of my knowledge. Ready for testing!
| Labels |
Added:
NPM Resource Changed
|
||
Please fix conflicts.
| Labels |
Added:
Conflicting Files
|
||
This pull request has been automatically rebased to 5.0-dev.
Hey @HLeithner and @Hackwar , this is a bug fix regarding access/permissions issues, not a feature. Is it possible to move this back to 4.3? It's pretty much ready though I see now I need to check why the build is failing.
I rebased it to 4.3-dev, not changing the label, RM for 4.3 should update the label if it's a bug.
Thank you Harald! @obuisard and @sdwjoomla , what are your thoughts?
Crystal @crystalenka, I think that is a UI/UX bug, so I would like to see it carried through 4.3.
However, we will need to adjust the code to rid of the conflicts
Agree with Olivier this is a bug as it does not function as expected
Crystal @crystalenka can you look at the conflicts? Let us know if you need help with that. Thanks!
Thank you! It's on my list and I will check it as soon as I can.
No pressure Crystal @crystalenka, just want to make sure you did not forget about it :-) Would love to see the fix in 4.3.3.
| Labels |
Added:
bug
PR-4.3-dev
Removed: ? |
||
Hi @obuisard , sorry for taking so long. I believe I fixed the conflicts now.
Thank you very much Crystal @crystalenka. That will help get it tested :-)
| Labels |
Removed:
Conflicting Files
|
||
Log in as a non super user.
Edit an article.
See console.
| Labels |
Added:
?
|
||
This pull request has been automatically rebased to 4.4-dev.
| Title |
|
||||||
This pull request has been automatically rebased to 5.2-dev.
| Title |
|
||||||
This is as far as I can get it with my own skills. I'm not sure why the media field in disabled subforms breaks things so astonishingly.
Here's what I'm experiencing:
Invalid field:[Media field name]