User tests: Successful: Unsuccessful:
This is the same as #1192 but for staging instead of master.
Labels |
Added:
?
|
Labels |
Added:
?
|
Labels |
Added:
?
|
Fix some merge conflicts in this old thing. This is still really cool, btw.
Hmm looks like a unit test fails here?
Can you have a look? https://travis-ci.org/joomla/joomla-cms/builds/82667974
There was a bug in my code but I also found that the unit test was laughable. So it's a bit better now. However, it's using the assertTag
test which may be a problem in the future. I think it's the best test for html because it doesn't fail on things like whitespace or other things that shouldn't matter in html but it seems to be removed in later version of phpunit
so we may need to find a new way to test html output.
It's removed from PHPUnit 5. So either we need to eventually write something similar or pull in some third party library that has the ability to parse that stuff in a unit test run.
I'd rather just see all html removed from server-side code but that's kind of a pipe dream.
@okonomiyaki3000 can you delete this file: libraries/joomla/form/options/index.html? Joomla dropped all the blank .html files long time ago
Yeah, I seem to have some vague recollection of that.
Category | ⇒ | Fields |
Labels |
@okonomiyaki3000 I know this is another old PR but I agree that this is still cool to implement. If you could update the PR to resolve the merge conflicts I will get some people to test this and we can implement it for 3.6. Thanks.
Milestone |
Added: |
@okonomiyaki3000 I have set the milestone for now to 3.7.0 as we are too late for 3.6.0.
Category | Fields | ⇒ | Fields Unit Tests |
Labels |
I have tested this item
I tested this Patch with the Patch Tester 2.0.1 and it throws an error: "The file marked for modification does not exist: layouts/joomla/fields/radio.php"
Tested @icampus
I have tested this item
Same error as Edu20 - file missing. Does this need another patch before?
@icampus
Make sure you've actually got that file in your local install. Should be present as of 3.5.0.
Oh, nevermind, I see the issue now. This does need to be sync'd with staging it looks like; the file it's groaning about appears to have been renamed.
Sorry guys, this is really old and needs a lot of work. I'm glad it's getting some attention again because I still think it's a great idea but I need to find some time to to fix it up.
Also, I never took into account grouped lists. Really I don't know why grouped lists need to be any different from ungrouped lists. I'll need to have that in mind when updating this I guess.
@okonomiyaki3000 Yeah I know this is old but you know by now that I tend to drag these things out into the light ;) Shall we close this issue for now and can be re-opened or a new one can be made when you have time for it?
Status | Pending | ⇒ | Information Required |
Labels |
Status | Information Required | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-10-03 14:40:24 |
Closed_By | ⇒ | zero-24 | |
Labels |
I have opend a new PR maybe you can fork that branch and do your own PR?
OK, that sounds good. has some work been done on it then? I guess I'd better go see.
https://github.com/zero-24/joomla-cms/tree/5147
I have just fixed the conflicts
No there is no additional work done.
Can you fix the codestyle errors?