? bug PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
2 Apr 2023

Pull Request for Issue #40262

Summary of Changes

Set clicks and impmade to 0 when a banner is saved as copy

Testing Instructions

Copy a banner and check that values for clicks and impressions are set to 0

Actual result BEFORE applying this Pull Request

see #40262

Expected result AFTER applying this Pull Request

clicks and impressions for a ccopy is 0

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

e4d3c97 2 Apr 2023 avatar chmst cs
avatar joomla-cms-bot joomla-cms-bot - change - 2 Apr 2023
Category Administration com_banners
avatar chmst chmst - open - 2 Apr 2023
avatar chmst chmst - change - 2 Apr 2023
Status New Pending
avatar chmst chmst - change - 3 Apr 2023
Title
Reset banner statistics in copy
[4.3] Reset banner statistics in copy
avatar chmst chmst - edited - 3 Apr 2023
avatar viocassel viocassel - test_item - 3 Apr 2023 - Tested successfully
avatar viocassel
viocassel - comment - 3 Apr 2023

I have tested this item successfully on e4d3c97


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

avatar richard67 richard67 - test_item - 4 Apr 2023 - Tested unsuccessfully
avatar richard67
richard67 - comment - 4 Apr 2023

I have tested this item ? unsuccessfully on e4d3c97

When using the "Save as Copy" button in the edit view, the PR works well.

But when using batch copy in the list view to copy a banner to another category, the statistics are not reset.


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

avatar chmst chmst - change - 8 Apr 2023
Labels Added: bug PR-4.3-dev ?
avatar chmst
chmst - comment - 8 Apr 2023

Clicks and Impressions are now initialised in batchCopy.

Open: Is this B/C break?

If the client is changed, clicks and impressions remain unchanged. Correct?

_

avatar chmst chmst - change - 9 Apr 2023
Labels Added: ?
avatar richard67 richard67 - change - 9 Apr 2023
Labels Removed: ?
avatar richard67 richard67 - test_item - 9 Apr 2023 - Tested successfully
avatar richard67
richard67 - comment - 9 Apr 2023

I have tested this item successfully on 450d987

I've tested both the "save & copy" button in the edit view and the batch copy of one and of multiple banners to another category.

In all cases the clicks and impressions were reset.


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

avatar richard67
richard67 - comment - 9 Apr 2023

@viocassel Could you test again, this time also the batch copy of banners to another category? Thanks in advance.

avatar viocassel viocassel - test_item - 9 Apr 2023 - Tested successfully
avatar viocassel
viocassel - comment - 9 Apr 2023

I have tested this item successfully on 450d987

@richard67 ?


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

avatar richard67 richard67 - change - 9 Apr 2023
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 9 Apr 2023

RTC


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

avatar sdwjoomla sdwjoomla - change - 5 May 2023
Labels Added: ?
avatar obuisard obuisard - change - 5 May 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-05 23:39:21
Closed_By obuisard
avatar obuisard obuisard - close - 5 May 2023
avatar obuisard obuisard - merge - 5 May 2023
avatar obuisard
obuisard - comment - 5 May 2023

Thank you Christiane @chmst for the PR :-)

Add a Comment

Login with GitHub to post a comment