? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
2 Aug 2017

Pull Request for Issue #17380 .

Summary of Changes

If you select a category in Searchtools the categories are displayed like that:
image

If you select it in the Article, the Parent item has already a dash in front:
image

So in one case its:
Category Parent

  • category child

and in the other case

  • Category Parent
    -- category child

UPDATED

I have adjusted the select box in the Category select

Test instructions

  1. Create a main category and sub category
  2. Observe that in the category select for a new item they are all displayed with at least one - dash
  3. Apply the PR and observe that parents no longer have a - dash
  4. Unpublish categories and observe that they are still wrapped in [] eg [unpublished category]
avatar joomla-cms-bot joomla-cms-bot - change - 2 Aug 2017
Category Libraries
avatar brianteeman brianteeman - open - 2 Aug 2017
avatar brianteeman brianteeman - change - 2 Aug 2017
Status New Pending
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 2 Aug 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Aug 2017

I have tested this item successfully on 4af9648


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

avatar infograf768
infograf768 - comment - 2 Aug 2017

Imho, the display in the SearchTools is correct, not the one in categoryedit field type

avatar brianteeman
brianteeman - comment - 2 Aug 2017

ok - i will switch it around

avatar brianteeman brianteeman - change - 2 Aug 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-08-02 09:50:23
Closed_By brianteeman
Labels Added: ?
avatar brianteeman brianteeman - close - 2 Aug 2017
avatar infograf768
infograf768 - comment - 2 Aug 2017

I solved it here by adding

			if ($options[$i]->level != 0)
			{
				$options[$i]->level = $options[$i]->level -1;
			}

in categoryedit.php line 245

avatar infograf768
infograf768 - comment - 2 Aug 2017

the reason is that the same type is used for com_categories parent and articles categories.

avatar brianteeman
brianteeman - comment - 2 Aug 2017

Thanks I have already made the change and will commit it this afternoon when I get home from a funeral. Will also add indents to the tag search tool filter as well

avatar brianteeman brianteeman - change - 3 Aug 2017
The description was changed
avatar brianteeman brianteeman - edited - 3 Aug 2017
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2017
Category Libraries Administration com_categories
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2017
Category Administration com_categories
avatar brianteeman
brianteeman - comment - 3 Aug 2017

Updated PR and test instructions

avatar brianteeman brianteeman - change - 3 Aug 2017
Status Closed New
Closed_Date 2017-08-02 09:50:23
Closed_By brianteeman
avatar brianteeman brianteeman - change - 3 Aug 2017
Status New Pending
avatar brianteeman brianteeman - reopen - 3 Aug 2017
avatar infograf768 infograf768 - test_item - 3 Aug 2017 - Tested successfully
avatar infograf768
infograf768 - comment - 3 Aug 2017

I have tested this item successfully on 132e955


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 3 Aug 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Aug 2017

I have tested this item successfully on 132e955


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Aug 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Aug 2017

RTC after two successful tests.

avatar mbabker mbabker - change - 3 Aug 2017
Labels Added: ?
avatar mbabker mbabker - change - 3 Aug 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-03 16:51:22
Closed_By mbabker
avatar mbabker mbabker - close - 3 Aug 2017
avatar mbabker mbabker - merge - 3 Aug 2017
avatar brianteeman
brianteeman - comment - 4 Aug 2017

Thanks

Add a Comment

Login with GitHub to post a comment