? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
18 Aug 2018

Pull Request for Issue #11976 .

Summary of Changes

Moves the overrides view to use search tools

Known Bug that needs solving before this is merged

I can't seem to get the session to persist the client across into the edit view - so it always uses the default (administrator) - filter by english admin then create a new overrides and the client is site not admin. @mbabker would appreciate some help cause I'm sure i'm doing something crazy dumb in the session

Testing Instructions

After patch we have search tools instead of the filtering in the sidebar

Documentation Changes Required

help screens

//cc @infograf768

avatar wilsonge wilsonge - open - 18 Aug 2018
avatar wilsonge wilsonge - change - 18 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Aug 2018
Category Administration com_languages
avatar wilsonge wilsonge - change - 18 Aug 2018
The description was changed
avatar wilsonge wilsonge - edited - 18 Aug 2018
avatar wilsonge wilsonge - change - 18 Aug 2018
The description was changed
avatar wilsonge wilsonge - edited - 18 Aug 2018
avatar dgrammatiko
dgrammatiko - comment - 18 Aug 2018

@wilsonge why don't you just merge #19552 or the other PR? I think he already patched this there

avatar wilsonge
wilsonge - comment - 18 Aug 2018

That PR removes the sidebar from com_content. This is just moving the search functionality into searchtools in com_languages overrides. There's literally no common code between these two prs?

avatar dgrammatiko
dgrammatiko - comment - 18 Aug 2018

@wilsonge I meant this PR: #19590

avatar brianteeman
brianteeman - comment - 18 Aug 2018

@dgrammatiko well for a start this PR is for Joomla 3 and the one you link to is for Joomla 4

avatar wilsonge wilsonge - change - 18 Aug 2018
Labels Added: ?
avatar wilsonge
wilsonge - comment - 18 Aug 2018

and that one is doing extra things and is closed ;)

avatar brianteeman
brianteeman - comment - 20 Aug 2018

I have tested this item successfully on bc163ea


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

avatar brianteeman brianteeman - test_item - 20 Aug 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 20 Aug 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Aug 2018

I have tested this item successfully on bc163ea


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Aug 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Aug 2018

Ready to Commit after two successful tests.

avatar mbabker mbabker - change - 3 Sep 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-09-03 17:14:42
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 3 Sep 2018
avatar mbabker mbabker - merge - 3 Sep 2018
avatar infograf768
infograf768 - comment - 4 Sep 2018

Sorry to say but I am not so happy with this
One has to click on search tools to define for which language and site or admin to make the override.
It will always default to the alpha order of the language (and site) if not selected through searchTools.

We have to find a better solution imho.

EDIT: I mean we may still use searchtools to filter (although we need a solution to display all overrides), but we need a specific field separated from SearchTools to select to which language and client we want to create a new override

avatar brianteeman
brianteeman - comment - 4 Sep 2018

Like the one in modules to select site or admin ??

image

avatar infograf768
infograf768 - comment - 4 Sep 2018

Similar, yes. Rather like in menu items manager for Select Menu.

Basically, this means, if we want to do this right, some changes for the creation/editing of a new override as it is there imho that the language/client has normally to be selected, as we do for menu items where the Menu, if not already selected in the filter, has to be selected in the menu field

avatar infograf768
infograf768 - comment - 4 Sep 2018

or also in com_associations

avatar brianteeman
brianteeman - comment - 4 Sep 2018

Should be easy enough - but as this has been merged please open a new issue for it

avatar infograf768
infograf768 - comment - 4 Sep 2018

Not so easy.

avatar infograf768
infograf768 - comment - 5 Sep 2018

See #22014

Add a Comment

Login with GitHub to post a comment