? Pending

User tests: Successful: Unsuccessful:

avatar JoomliC
JoomliC
28 Nov 2018

Pull Request for Issue with com_banners Clients published state broken in 3.9.1
Related to PR #22851

Summary of Changes

  • Add column alias

Testing Instructions

  • Test to change published state for one or multiple Clients with button and checkbox.
  • Apply this PR and test again.

Expected result

  • Published state is changed
    capture d ecran 2018-11-28 a 17 16 29

Actual result

  • No effect
    capture d ecran 2018-11-28 a 17 16 59
avatar JoomliC JoomliC - open - 28 Nov 2018
avatar JoomliC JoomliC - change - 28 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Nov 2018
Category Administration com_banners
avatar Quy
Quy - comment - 28 Nov 2018

I have tested this item successfully on 9092012


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

avatar Quy Quy - test_item - 28 Nov 2018 - Tested successfully
avatar laoneo
laoneo - comment - 29 Nov 2018

Before testing this one, better to wait for a decision made in #23197.

avatar SharkyKZ
SharkyKZ - comment - 30 Nov 2018

@laoneo this needs to be merged regardless of whether #22851 is reverted or not.

avatar infograf768
infograf768 - comment - 30 Nov 2018

I have tested this item successfully on 9092012


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

avatar infograf768 infograf768 - test_item - 30 Nov 2018 - Tested successfully
avatar infograf768 infograf768 - change - 30 Nov 2018
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 30 Nov 2018

rtc


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

avatar mbabker mbabker - change - 1 Dec 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-12-01 18:04:22
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 1 Dec 2018
avatar mbabker mbabker - merge - 1 Dec 2018

Add a Comment

Login with GitHub to post a comment