Cleanup batch copy of items

[wilsonge]

Summary of Changes

Cleanup of the batch copy'ing of data. This introduces a empty post processing of batch copied data in order for com_content to add items into the frontpage table (and other components to do whatever processing they want).

Testing Instructions

Test Banners, Newsfeeds and contact work as is (there are no changes here - they just implement the parent method which is the same)

In com_content test batch copying with items that are featured and aren't featured and ensure that items are created correctly in the #__content_frontpage table

Documentation Changes Required

None

[webnet-assmann]

[joomlamarco]

I have tested this item 🔴 unsuccessfully on 44f856d

Banner creates an error.News, Contact works fine. The entries into table #_content_frontpage are correct.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13501.}

[wilsonge]

Hey both. can you please tell me what errors you are getting. I can't seem to reproduce anything at the moment (I have merged in staging since you both tested so it might be simply related to that)

[webnet-assmann]

See the Screenshot, i've uploaded

[wilsonge]

Hmm I definitely can't reproduce. Can you please try again on a clean copy of staging/nightly. Maybe just merging in staging now fixed something?

[webnet-assmann]

Reproduced it on a nightly build - same error. It's localhost with xampp on windows10 php7.0.9

[webnet-assmann]

Feel free to contact @zero-24

[zero-24]

Hmm I definitely can't reproduce. Can you please try again on a clean copy of staging/nightly. Maybe just merging in staging now fixed something?

@wilsonge it is about the different sql definition https://github.com/joomla/joomla-cms/blob/staging/installation/sql/mysql/joomla.sql#L111 for banners the title field is called name so we need a extra case for that 😄

[wilsonge]

Umm i don't understand how I can't reproduce this cause clearly you are correct. But I guess this latest version should fix things?

[webnet-assmann]

I have tested this item ✅ successfully on 1121efe

Everything is fine now :-)

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13501.}

[wilsonge]

Merged in staging

[roland-d]

@wilsonge Is this merged?? If not, can you update the merge conflicts?

[priiish]

I have tested this item 🔴 unsuccessfully on 2952f2a

- Tried to copy banner to another category via batch feature

• error coming up:
  "Argument 1 passed to Joomla\CMS\MVC\Model\AdminModel::cleanupPostBatchCopy() must be an instance of Joomla\CMS\MVC\Model\JTableInterface, instance of BannersTableBanner given, called in
  <path-to-joomla!>/Joomla_3/libraries/src/MVC/Model/AdminModel.php on line 450
  <path-to-joomla!>/Joomla_3/libraries/src/MVC/Model/AdminModel.php:472"

therefore tested unsuccessful

@icampus

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13501.}

[brianteeman]

I have tested this item ✅ successfully on 7ec4d5d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13501.}