? Success

User tests: Successful: Unsuccessful:

avatar pe7er
pe7er
18 May 2015

This PR adds new pre-selection feature in the edit form of the Category Manager: The form fields Status, Language and Access Level are selected on basis of the [Search Tools] Set Filters in the Category Manager. Similar PR as #6966.

Test instructions

Test in Category Manager: Articles, Category Manager: Banners, Category Manager: Contacts, Category Manager: News Feeds if the selected [Search Tools] > **Filters (Status, Language and Access Level) are pre-selected in the form when creating a New Category

Category Manager: Articles

Back-end > Content > Category Manager > [Search Tools] > select (Status, Language and Access Level).

screen shot 2015-05-18 at 04 34 06

Create new Category and check if the parameters on the left are "pre-selected".

screen shot 2015-05-18 at 04 34 07

Category Manager: Banners

Back-end > Components > Banners > Categories > [Search Tools] > select (Status, Language and Access Level).

screen shot 2015-05-18 at 04 34 06

Create new Category and check if the parameters on the left are "pre-selected".

screen shot 2015-05-18 at 04 34 07

Category Manager: Contacts

Back-end > Components > Contacts > Categories > [Search Tools] > select (Status, Language and Access Level).

screen shot 2015-05-18 at 04 34 07

Create new Category and check if the parameters on the left are "pre-selected".

screen shot 2015-05-18 at 04 34 07

Category Manager: News Feeds

Back-end > Components > News Feeds > Categories > [Search Tools] > select (Status, Language and Access Level).

screen shot 2015-05-18 at 04 34 07

Create new Category and check if the parameters on the left are "pre-selected".

screen shot 2015-05-18 at 04 34 06

avatar pe7er pe7er - open - 18 May 2015
avatar pe7er pe7er - change - 18 May 2015
The description was changed
avatar zero-24 zero-24 - change - 18 May 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 18 May 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 18 May 2015
Category Administration UI/UX
avatar zero-24 zero-24 - change - 18 May 2015
Status New Pending
Easy No Yes
avatar sovainfo sovainfo - test_item - 18 May 2015 - Tested successfully
avatar ketchupmonki ketchupmonki - test_item - 20 May 2015 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 May 2015

@test: works as described.

Pre-select Fields in Category-Manager of Articles, Banners, Contacts and News Feeds are same Fields in "New" Article, Banner, Contact and News Feed.

avatar zero-24 zero-24 - alter_testresult - 26 May 2015 - franz-wohlkoenig: Tested successfully
avatar zero-24 zero-24 - change - 26 May 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 26 May 2015

As we have successful tests by @sovainfo @franz-wohlkoenig and @ketchupmonki I mark this as RTC Thanks :smile:

PS: @sovainfo @ketchupmonki can you please send a simple comment (like: @test successful) if you test? Else we don't get a notification so we don't know that you tested this :smile: Thanks. :+1:


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

avatar zero-24 zero-24 - change - 26 May 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 26 May 2015
Labels Added: ?
avatar zero-24 zero-24 - close - 30 May 2015
avatar roland-d roland-d - change - 30 May 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-05-30 13:29:51
Closed_By roland-d
avatar roland-d roland-d - close - 30 May 2015
avatar roland-d roland-d - close - 30 May 2015
avatar infograf768
infograf768 - comment - 31 May 2015

@roland-d @wilsonge
Again the same error:
$data->set('language', $app->input->getVar('language', (isset($filters['language']) ? $filters['language'] : null)));

Please do not merge any of that serie of PRs without correcting to
$data->set('language', $app->input->getString('language', (isset($filters['language']) ? $filters['language'] : null)));

and correct this one.

Thanks

avatar wilsonge
wilsonge - comment - 31 May 2015

I did straight after Roland merged it - 07af341

avatar zero-24 zero-24 - change - 31 May 2015
Milestone Added:
avatar smz
smz - comment - 11 Jun 2015

I'm quite sure this created the regression evidenced in #7151...

avatar woluweb
woluweb - comment - 12 Jun 2015

Hi @pe7er,

Success !

I have just tested on a site having the following four patches activated
6938 Added countItems to com_contact helper file GitHub J! Issue Applied Revert Patch
6936 Category Manager News Feeds count GitHub J! Issue Applied Revert Patch
6934 Category manager - banner item count
6916 Added Article count (published, unpublished, trashed) to Category Man…

I have first installed French language.

Then I tested as required the four category managers

  • articles
  • contact
  • banners
  • newsfeed

Every test was successfull. Txs

avatar johanjanssens johanjanssens - reference | f7a2002 - 19 Jun 15
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?
avatar pe7er pe7er - head_ref_deleted - 5 Nov 2015

Add a Comment

Login with GitHub to post a comment