? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
28 Nov 2021

This PR removes a conditional block from the article manager and the featured article manager that was intended to ensure that the display of the category and nested category was correct when the admin is using an rtl language and the category names may be a mix of LTR and RTL.

Not really sure why but

  1. this code does not work correctly. the order of the categories is wrong and the << character is wrong
  2. there is no need for this alternate block of code for RTL

Before

image

After

image

avatar brianteeman brianteeman - open - 28 Nov 2021
avatar brianteeman brianteeman - change - 28 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Nov 2021
Category Administration com_content
avatar brianteeman brianteeman - change - 28 Nov 2021
Labels Added: ?
avatar brianteeman brianteeman - change - 28 Nov 2021
The description was changed
avatar brianteeman brianteeman - edited - 28 Nov 2021
avatar ceford
ceford - comment - 29 Nov 2021

For some reason, with or without the patch applied, with Persian as the admin language I see Persian text embedded in the Category name. also containing a colon:

screen shot 2021-11-29 at 19 00 08


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

avatar ceford
ceford - comment - 29 Nov 2021

For some reason, with or without the patch applied, with Persian as the admin language I see Persian text embedded in the Category name. also containing a colon:

screen shot 2021-11-29 at 19 00 08


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

avatar brianteeman
brianteeman - comment - 29 Nov 2021

thats the word category in persian

avatar infograf768
infograf768 - comment - 30 Nov 2021

I do not get the same result as the screenshots above. Never gets 2 « before or after patch.

Missing instructions: what is the sub category to which is assigned the article in the original description ?

In the sample below, the parent is Kategorie(de-de) and I created a subcategory in which I moved the article.

BEFORE PATCH , RTL

Screenshot 2021-11-30 at 09 01 47

AFTER PATCH
RTL
Screenshot 2021-11-30 at 08 49 30

LTR

Screenshot 2021-11-30 at 08 57 11

Is it what is expected? If yes, patch looks good.

avatar brianteeman
brianteeman - comment - 30 Nov 2021

I am guessing that your categories are

Kategorie (de-de)
-  مجموعه whatever (fa-ir)

If so then your screenshots show the expected results and a faulty before shot

Not sure what you mean about a second «

avatar brianteeman
brianteeman - comment - 14 Dec 2021

@infograf768 please can you respond

avatar brianteeman
brianteeman - comment - 18 Jan 2022

@infograf768 please explain what the problem is as your comment is blocking this being merged

avatar infograf768
infograf768 - comment - 19 Jan 2022

Read again:
Is it what is expected? If yes, patch looks good.

I just remarked a wrong description of the issue. You wrote:
this code does not work correctly. the order of the categories is wrong and the << character is wrong

instead of and the « character is wrong

avatar brianteeman
brianteeman - comment - 19 Jan 2022

Ah you're playing your silly games again. Never mind I will just close it.

avatar brianteeman brianteeman - change - 19 Jan 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-01-19 08:14:04
Closed_By brianteeman
avatar brianteeman brianteeman - close - 19 Jan 2022
avatar brianteeman brianteeman - change - 20 Apr 2022
Status Closed New
Closed_Date 2022-01-19 08:14:04
Closed_By brianteeman
Labels Removed: ?
avatar brianteeman brianteeman - change - 20 Apr 2022
Status New Pending
avatar brianteeman brianteeman - reopen - 20 Apr 2022
avatar brianteeman brianteeman - change - 20 Apr 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-04-20 17:03:14
Closed_By brianteeman
Labels Added: ?
avatar brianteeman brianteeman - close - 20 Apr 2022

Add a Comment

Login with GitHub to post a comment