User tests: Successful: Unsuccessful:
Labels |
Added:
?
|
@test Tested this successfully.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.
The issue is related to the usage of JToolBar::appendButton() function in com_media as there is no such boolean parameter available(true or false) which if passed can generate an alert box asking for making a selection from the list. Where as in case of other pre-defined JToolBar buttons, like publish , unpublish etc. the alert popup can be easily implemented by simply passing the boolean parameter.
The important point in this case which needs to be enlightened is that the delete button in this issue is not the pre-defined JToolbarHelper::deleteList(); but it is a custom JToolBar::appendButton(); button. So, the work around for this would be customizing the javascript code in the component but not in the library files.I am working on it.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.
Title |
|
Title |
|
Category | ⇒ | Media Manager |
Based on the last comment I am setting this one as Needs Review by maintainers
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.
Status | Pending | ⇒ | Needs Review |
The solution for this one seems to be ok (and I was able to test it successfully as well).
The current Travis CI build failure being shown on the Pull Request page for this one seems to be unrelated to the actual patch itself.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.
I was able to reproduce the bug: in Media Manager, click the Delete button with nothing selected. It reloads like it's doing something with no error or confirmation of what happened. After applying the patch, tested again and clicking Delete button without anything selected results in error box "Please first make a selection from the list". Bug fixed.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.
There is an error in the unittests with your changes. Please see if you can fix those.
Status | Needs Review | ⇒ | Pending |
Easy | No | ⇒ | Yes |
I can't see anything wrong with this - removing from needs review. Can I have one test here to confirm the patch is still required and works as expected before it goes RTC please?
@wilsonge is the comment about unit tests no longer valid either?
I don't think so. There are no unit tests failing with this and there are none covering this kinda stuff anyhow. I think Dmitris will throw a small hissy fit about using script tags rather than using JDocument but given the age of this PR i"d rather merge this if it works and then get him to do that separately
I have tested this item successfully on 97f89dc
I have tested this item successfully on 97f89dc
For consistency this is correct.
Personally this type of functionality should be labelled "you are an idiot why did you click without selecting"
Status | Pending | ⇒ | Ready to Commit |
RTC
Not a new feature as this functionality is present everywhere else so setting 3.5.2 milestone
Labels |
Added:
?
|
Milestone |
Added: |
@rbsl-rimjhim could you check the merge conflicts, thanks
Robert as this PR is 18 months old think this is one of those cases where we can do it ourselves ;)
Feel free to do it :-P
I will do it now
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-13 11:21:31 |
Closed_By | ⇒ | brianteeman | |
Labels |
Labels |
See #9891
Status | Closed | ⇒ | New |
Closed_Date | 2016-04-13 11:21:31 | ⇒ | |
Closed_By | brianteeman | ⇒ |
Status | New | ⇒ | Pending |
Labels |
Removed:
?
|
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-13 11:29:40 |
Closed_By | ⇒ | wilsonge |
Milestone |
Milestone |
Added: |
Milestone |
Added: |
Milestone |
Confirmed. I get a message now :) http://screencast.com/t/XJGLzlTVj
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4777.