? ? ? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
7 Jun 2017

Pull Request for Issue # .

Summary of Changes

Bring back the missing strings

Testing Instructions

Apply patch, check com_media's options

Expected result

screen shot 2017-06-07 at 16 06 49

Actual result

screen shot 2017-06-07 at 16 02 11

Documentation Changes Required

avatar dgt41 dgt41 - open - 7 Jun 2017
avatar dgt41 dgt41 - change - 7 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jun 2017
Category Administration Language & Strings
avatar dgt41 dgt41 - change - 7 Jun 2017
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 7 Jun 2017

I have tested this item successfully on 386d901


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

avatar brianteeman brianteeman - test_item - 7 Jun 2017 - Tested successfully
avatar laoneo
laoneo - comment - 7 Jun 2017

I have tested this item successfully on 386d901


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

avatar laoneo
laoneo - comment - 7 Jun 2017

Thanks


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

avatar laoneo laoneo - test_item - 7 Jun 2017 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 7 Jun 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 7 Jun 2017

RTC


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

avatar brianteeman
brianteeman - comment - 8 Jun 2017

I have tested this item ? unsuccessfully on 386d901

reverting my successful test as this string is still missing
COM_MEDIA_CONFIGURATION="Media: Options"


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16561.
avatar brianteeman brianteeman - test_item - 8 Jun 2017 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 8 Jun 2017

Please remove RTC for now


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

avatar dgt41 dgt41 - change - 8 Jun 2017
Labels Added: ?
avatar dgt41
dgt41 - comment - 8 Jun 2017

@brianteeman just added it back, is there anything else missing here?

avatar brianteeman
brianteeman - comment - 8 Jun 2017

Thanks for that one. As for other strings I have no idea because it is completely unfunctional so I can't use it to see

avatar wilsonge wilsonge - change - 8 Jun 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-08 10:53:15
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 Jun 2017
avatar wilsonge wilsonge - merge - 8 Jun 2017
avatar wilsonge
wilsonge - comment - 8 Jun 2017

What do you mean by unfunctional?

avatar brianteeman
brianteeman - comment - 8 Jun 2017

@wilsonge all i see is this screenshot and no buttons are functional
screenshotr12-58-05

and the following js error in the console

Uncaught TypeError: Cannot read property 'filter' of undefined
    at Api._normalizeArray (mediamanager.js:8763)
    at Object.<anonymous> (mediamanager.js:8633)
    at i (jquery.min.js:2)
    at Object.fireWith [as resolveWith] (jquery.min.js:2)
    at A (jquery.min.js:4)
    at XMLHttpRequest.<anonymous> (jquery.min.js:4)
avatar laoneo
laoneo - comment - 8 Jun 2017

Did you do a fresh install? If not then you need to discover the new media manager plugins.

avatar Bakual
Bakual - comment - 8 Jun 2017

Did you do a fresh install? If not then you need to discover the new media manager plugins.

I can confirm that after discovering and enabling the plugins the console errors are gone.
I opened a new issue about that as I think the UX needs to be improved here.

avatar brianteeman
brianteeman - comment - 8 Jun 2017

the plugins were already enabled - it was a new install
screenshotr18-20-41

avatar brianteeman
brianteeman - comment - 8 Jun 2017

screenshotr18-21-56

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 9 Jun 2017

On a fresh installed 4.0-dev explored Plugins and activate them:
bildschirmfoto 2017-06-09 um 06 40 22
got this, seems fine:
bildschirmfoto 2017-06-09 um 06 41 52

avatar wilsonge
wilsonge - comment - 9 Jun 2017

@laoneo do you want to get in touch with @brianteeman and try and work this through as it would appear to be something specific to his setup?

avatar laoneo
laoneo - comment - 9 Jun 2017

I asked in the j4 geoup if somebody with a windows environment can have a look as I can't repeoduce the issue.

avatar joomdonation
joomdonation - comment - 9 Jun 2017

I just installed latest 4.0-dev and it works OK. Tried to navigate between folder, create folder, upload images... and it works (just a quick try only)

I also don't see any error in the console. Look like it works fine. I am using Windows 8.

avatar Bakual
Bakual - comment - 9 Jun 2017

I think it is related somehow to the special setup Brian has. It is a "real" Windows server, not the wanna-be XAMPP type one we usually test on :)

avatar brianteeman
brianteeman - comment - 9 Jun 2017

Which is even more reason to try and work out what the issue is. It might be an apache or php module that needs to be enabled. If that is the case then it needs to be added to the requirements etc

Add a Comment

Login with GitHub to post a comment