[#12453] - JFormOption
- Closed
- 22 Jul 2018
- Medium
- Build: staging
- # 12453
- Diff
- okonomiyaki3000:JFormOption
- Success continuous-integration/drone the build was successful Details
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
This is a revival of #5147 which was really just a continuation of #1192 so please see those for historical discussions.
The basic idea here is to eliminate the need for the many different field types that generate specific kinds of lists while, at the same time, making list, checkboxes, and radiobutton types more flexible and consistent. This is done with the concept of option types. So that now the the <option> tag can take a type attribute (and probably other attributes as needed) which will determine which options are to appear in the select, radio, checkbox or other form control.
Testing Instructions
Create a form with fields using the list, radio, or checkboxes type and assign it options of one or more of the option types....
OK, that's a big hassle. I think what this pr really needs is for these option types to actually be used in the places where they can be used. That is coming soon.
Documentation Changes Required
Certainly.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
?
|
||
| Category | ⇒ | Layout Libraries Unit Tests |
| Title |
|
||||||
| Title |
|
||||||
| Labels |
Removed:
?
|
||
This should fix the failing tests but now it looks like I need to rebase.
this should fix those conflicts but I had some errors and failures in the tests. only in tests that had nothing at all to do with this PR though. So whatever that means...
OK, there was one legit error there that was probably mine. Let's see if it works now.
| Category | Layout Libraries Unit Tests | ⇒ | Administration com_admin com_config com_contact com_languages com_users Front End com_mailto Installation Layout Libraries |
@okonomiyaki3000 Do you want to resolve the merge conflicts or close this?
I hope we can get this one in J4, the code so much better than the current codebase
Hmm. I still want to do this but I think it might be easier to start a new branch instead of trying to rebase this one.
@okonomiyaki3000 Should we tag this for Joomla 4?
I just need to redo the whole thing for Joomla 4.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-07-22 23:53:12 |
| Closed_By | ⇒ | okonomiyaki3000 |
I'll look at some of these failing tests soon and also add some new tests but this latest commit actually puts this feature to use in many of the standard forms. You can now test the usability by using any of these modified forms (for example, the Global Configuration form).