?

Success

Pull Request for # 7336

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
4 Jul 2015

Fixes #7336

The problem:

Go to com_media
select (using the checkboxes at the images) some images
try to hit the delete button.

Expected result
deleting images.

Actual result
Console error

Testing

Apply patch
follow the steps above
Verify that you get the expected result: images are trashed.

avatar dgt41 dgt41 - open - 4 Jul 2015
avatar dgt41 dgt41 - change - 4 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Jul 2015
Labels Added: ?
avatar jwaisner
jwaisner - comment - 4 Jul 2015

@test

PR resolves issue with the delete button not working.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7338.

avatar jwaisner jwaisner - test_item - 4 Jul 2015 - Tested successfully
avatar smz smz - test_item - 4 Jul 2015 - Tested successfully
avatar smz
smz - comment - 4 Jul 2015

#Test OK

Unrelated: I just noticed that folders containing blanks in their names are inaccessible by com_media, but I think I've seen this issue cited somewhere else...

avatar zero-24 zero-24 - change - 4 Jul 2015
Category JavaScript Media Manager
avatar zero-24 zero-24 - change - 4 Jul 2015
Status Pending Ready to Commit
Rel_Number 0 7336
Relation Type Pull Request for
Easy No Yes
avatar zero-24
zero-24 - comment - 4 Jul 2015

Thanks :smile: RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7338.

avatar zero-24 zero-24 - change - 4 Jul 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 4 Jul 2015
Milestone Added:
avatar zero-24
zero-24 - comment - 4 Jul 2015

@smz

Unrelated: I just noticed that folders containing blanks in their names are inaccessible by com_media, but I think I've seen this issue cited somewhere else...

This is expected behavior. You also can't create a folder with blanks ;)

avatar smz
smz - comment - 4 Jul 2015

@zero-24
TBH I agree that this can view as at least "enforcing best practices"

avatar zero-24 zero-24 - close - 4 Jul 2015
avatar wilsonge wilsonge - close - 4 Jul 2015
avatar wilsonge wilsonge - reference | a6a546b - 4 Jul 15
avatar wilsonge wilsonge - merge - 4 Jul 2015
avatar wilsonge wilsonge - change - 4 Jul 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-07-04 15:05:06
Closed_By wilsonge
avatar wilsonge wilsonge - close - 4 Jul 2015
avatar wilsonge
wilsonge - comment - 4 Jul 2015

Merged - thanks everyone :)

avatar dgt41 dgt41 - head_ref_deleted - 4 Jul 2015
avatar N6REJ
N6REJ - comment - 13 Jul 2015

is the pr gone once merged? Cause I can't find it in patch tester

avatar zero-24
zero-24 - comment - 13 Jul 2015

is the pr gone once merged?

Yes :smile:

avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment