? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
22 Nov 2018

Completing #23047 ++

Summary of Changes

Correcting RTL display of Joomla Update version in CPanel System as well as in the Joomla update component.
Correcting Extension version in Joomla Update component. That one required also to modify joomlaupdate default js file to change the order of version and 'Yes' when present

Basically we want to get for example:
4.0.0-alpha6-dev instead of alpha-dev-4.0.0 and align correctly the <span> concerned.

Testing Instructions

On last 4.0.0-dev branch
Modify libraries/src/Version.php
to const EXTRA_VERSION = 'alpha4-dev';
in order to get a database fix issue
Fix database.
Then in JoomlaUpdate Options choose Custom URL and add this link:
https://update.joomla.org/core/nightlies/next_major_list.xml

Install Persian language and set admin language to Persian.

Requires using npm ci to test.

Before patch

screen shot 2018-11-22 at 10 19 14

4 0jupdate_before_1

40jupdate_before_3

4 0jupdate_before_2

screen shot 2018-11-22 at 10 20 58

After patch

screen shot 2018-11-22 at 07 30 05
after-1
after-3
after2-a

If no update available

after2-b

screen shot 2018-11-22 at 09 18 12

Documentation Changes Required

None

@Quy @richard67 @C-Lodder @wilsonge

avatar infograf768 infograf768 - open - 22 Nov 2018
avatar infograf768 infograf768 - change - 22 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Nov 2018
Category Administration com_cpanel com_joomlaupdate Templates (admin) JavaScript Repository
avatar infograf768 infograf768 - change - 22 Nov 2018
Labels Added: ?
0a2243f 22 Nov 2018 avatar infograf768 cs
avatar richard67
richard67 - comment - 24 Nov 2018

I have tested this item successfully on 8db146a


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

avatar richard67 richard67 - test_item - 24 Nov 2018 - Tested successfully
avatar richard67
richard67 - comment - 24 Nov 2018

Fixes also display of version number of extensions like e.g. patchtester 3.0.0-beta4 in the pre-update check with RTL language.

In opposite to screenshot above, when using Persioan language during my test, the head line of the pre-update check was in English and already with correct version number in the "before patch" scenario. Maybe we used different versions of the Persioan language?


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

avatar infograf768
infograf768 - comment - 24 Nov 2018

@richard67
the string concerned in the persian 3.x lang pack contains some span html formatting (as all 3.x lang packs). i modified it here, which could explain.
thanks for testing!

avatar richard67
richard67 - comment - 24 Nov 2018

@infograf768 Welcome. Do you think you could have a look on my PR #23055 and maybe test or review or both?

avatar infograf768
infograf768 - comment - 24 Nov 2018

on it

avatar infograf768
infograf768 - comment - 24 Nov 2018

@wilsonge
I get this can now go in as RTL testers are rare... :)

avatar wilsonge wilsonge - close - 26 Nov 2018
avatar wilsonge wilsonge - merge - 26 Nov 2018
avatar wilsonge wilsonge - change - 26 Nov 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-11-26 15:43:53
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 26 Nov 2018

Thanks JM!

Add a Comment

Login with GitHub to post a comment