Conflicting Files ? ? Failure

User tests: Successful: Unsuccessful:

avatar florian1995
florian1995
8 Aug 2016

Pull Request for Issue #5474 .

I try to finish the great work from @jrseliga.
I fixed the problem that when you deselect for example all tags the list won't refresh. With the example code from @ggppdk it work's like a charm. Thanks for that.
I tested it with MySQL and PostgreSQL and it worked fine. It will be great if someone could test it also with MSSQL.

The following test instructions are also from @jrseliga.

Testing

Administrator

Articles

  1. Create 3 articles:
    1. Article 1 - Category 1 - Tag 1 - Author 1 - Public
    2. Article 2 - Category 2 - Tag 1 & Tag 2 - Author 2 - Registered
    3. Article 3 - Category 2 - Tag 2 - Author 1 - Super User
  2. View Articles
    1. Use Search Tools

Just with a simple structure like this, there is a large number of potential test cases. Please refer to the New behavior section for an overview of the filtering logic

Featured Articles

  1. View Featured Articles
    Use Search Tools

Categories

  1. View Categories
    Use Search Tools

Developed@icampus

avatar joomla-cms-bot joomla-cms-bot - change - 8 Aug 2016
Category Administration Components Language & Strings Front End Modules
avatar florian1995 florian1995 - open - 8 Aug 2016
avatar florian1995 florian1995 - change - 8 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Aug 2016
Labels Added: ? ?
avatar florian1995 florian1995 - change - 8 Aug 2016
The description was changed
avatar zero-24
zero-24 - comment - 8 Aug 2016

Hi @florian1995 i have fixed some Travis issues and a bug see: florian1995#1 👍

avatar florian1995 florian1995 - reference | ef32328 - 8 Aug 16
avatar florian1995
florian1995 - comment - 8 Aug 2016

Thank you really much @zero-24 for the quick response.
Now the code looks definitely better.
I added the missing semicolons and merged your code, now all build checks have passed, thank you :)

avatar zero-24
zero-24 - comment - 8 Aug 2016

👍

avatar ggppdk
ggppdk - comment - 10 Aug 2016

The labels for the multi-select fields are shown as placeholders

  • but when chosen attaches to the multi-select elements, the width of the input element is not calculated correctly, you need to click to open close the chosen element so that it is calculated correctly

To overcome the initial wrong inline CSS:

style="width: 0px;"

this CSS can be used as a workaround

ul.chosen-choices .search-field > input {
    width: 100% !important;
}

any better solution ?

avatar ggppdk
ggppdk - comment - 10 Aug 2016

I forgot to say that wrong width calculation happens only,

  • when the filters slider is closed

if you select any filter
then the slider is open on page load and the labels inside multi-select elements are shown correctly

appears_after_click

avatar florian1995
florian1995 - comment - 11 Aug 2016

Thanks for your reply @ggppdk.
If i understand you right the problem is, that sometimes the placeholders are not shown correctly, because the width isnt correct. But i tested it with Chrome and Firefox and dont know how to reproduce the issue, i can select, deselect, open and close the filters how i want, but it never happens. Maybe you can say in detail how to reproduce this issue.

avatar ggppdk
ggppdk - comment - 11 Aug 2016

Only thing i can say that the issue occurs (or can occur) when the selects are initially hidden,
because the slider is closed

There is a relevant issue still open in chosen tracker:
harvesthq/chosen#92

  • i am using select2 JS for my multi-select filters, and select2 JS in general in all my views, maybe if chosen JS had been updated , i would have been using it,

so i do not know much about it

i think suggested solution was to force it via CSS (see example above)

or add some JS code that .... e.g. will do something on slider open ...

avatar roland-d
roland-d - comment - 11 Aug 2016

@ggppdk Do you have any exact steps for reproducing the missing placeholder? I have tried to make it disappear but apparently I am not a magician. Thanks.

avatar roland-d roland-d - change - 11 Aug 2016
Milestone Added:
avatar zero-24 zero-24 - change - 3 Oct 2016
Milestone Removed:
avatar patjun
patjun - comment - 3 Jun 2017

testing


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

avatar patjun
patjun - comment - 3 Jun 2017

@ggppdk do you still have the problem with Joomla 3.7.2. I try to reproduce the problem, but i cant.
Please answer in the next days otherwise we will delete the issue. @nibra


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

avatar ggppdk
ggppdk - comment - 3 Jun 2017

I have not tested with j3.7.2 i guess if you tested and it now works , then it has been fixed

avatar Schmidie64 Schmidie64 - test_item - 21 Aug 2017 - Tested successfully
avatar Schmidie64
Schmidie64 - comment - 21 Aug 2017

I have tested this item successfully on 8a95b4a

@icampus
I've tested this Issue succesfull. Everything works fine.


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

avatar DrDreave DrDreave - test_item - 21 Aug 2017 - Tested successfully
avatar DrDreave
DrDreave - comment - 21 Aug 2017

I have tested this item successfully on 8a95b4a

Operating System

  • Joomla! 3.8.0-beta3-dev
  • PHP 5.6.2
  • MySQLi 5.5.38
  • Apache/2.2.29 (Unix)

Steps

  • Open article administration
  • Use search tools

Test before patch

  • Single filter selection
    Single filter selection

Test after patch

  • Multi filter selection for category, access, author tag
    Multi filter selection

Tested @icampus


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

avatar joomla-cms-bot joomla-cms-bot - change - 21 Aug 2017
Category Administration Components Language & Strings Front End Modules Administration com_categories com_content com_modules com_newsfeeds Language & Strings Front End Modules Components
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Aug 2017
The description was changed
Status Pending Ready to Commit
avatar joomla-cms-bot joomla-cms-bot - edited - 21 Aug 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Aug 2017

RTC after two successful tests.

Please resolve conflicting Files.

avatar THMmeddah THMmeddah - test_item - 22 Aug 2017 - Tested successfully
avatar THMmeddah
THMmeddah - comment - 22 Aug 2017

I have tested this item successfully on 8a95b4a

tested successfully


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

avatar THMmeddah
THMmeddah - comment - 22 Aug 2017

testing


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Aug 2017

@THMmeddah thanks for Test. We need 2 successfully Test for an Pull Request.

avatar AresRoxx10 AresRoxx10 - test_item - 22 Aug 2017 - Tested successfully
avatar AresRoxx10
AresRoxx10 - comment - 22 Aug 2017

I have tested this item successfully on 8a95b4a

Tested successfully


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Aug 2017

@AresRoxx10 thanks for Test. We need 2 successfully Test for an Pull Request, we have now 4 :-).

avatar roland-d
roland-d - comment - 28 Aug 2017

@franz-wohlkoenig Issue can be closed as implemented in #17668

avatar brianteeman brianteeman - close - 28 Aug 2017
avatar brianteeman brianteeman - change - 28 Aug 2017
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2017-08-28 16:06:40
Closed_By brianteeman
Labels Added: Conflicting Files ?
avatar brianteeman
brianteeman - comment - 28 Aug 2017

Closed as requested

Add a Comment

Login with GitHub to post a comment