? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
17 Nov 2019

The banners component has an alias field for the banner title. This is never used anywhere and is probably a result of copy pasting another component in the dim and distant past. It is crazy to maintain code that is not used so this PR removes the alias field.

Testing

  1. Create/edit banner
  2. save as new banner
  3. batch copy banners
  4. batch move banners

Pull Request for Issue #26843

avatar brianteeman brianteeman - open - 17 Nov 2019
avatar brianteeman brianteeman - change - 17 Nov 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Nov 2019
Category SQL Administration com_admin Postgresql com_banners Language & Strings Installation
avatar brianteeman brianteeman - change - 17 Nov 2019
Labels Added: ? ?
avatar Quy
Quy - comment - 17 Nov 2019

Category alias can be removed too.

avatar brianteeman
brianteeman - comment - 17 Nov 2019

Not really

avatar Quy Quy - test_item - 17 Nov 2019 - Tested successfully
avatar Quy
Quy - comment - 17 Nov 2019

I have tested this item successfully on 5bd61bb


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

avatar SharkyKZ
SharkyKZ - comment - 18 Nov 2019

@brianteeman Can you also update com_banners.banner record in #__content_types table?

avatar brianteeman
brianteeman - comment - 18 Nov 2019

do you mean in the field_mappings?

avatar SharkyKZ
SharkyKZ - comment - 18 Nov 2019

Yes.

avatar brianteeman
brianteeman - comment - 18 Nov 2019

@SharkyKZ done - i think

avatar brianteeman
brianteeman - comment - 28 Nov 2019

argue amongst yourselves - I can't be bothered

avatar brianteeman brianteeman - change - 28 Nov 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-11-28 19:03:24
Closed_By brianteeman
avatar brianteeman brianteeman - close - 28 Nov 2019
avatar richard67
richard67 - comment - 28 Nov 2019

Yes, ignorance is one of your key competences.
Sorry, that was not fair. Was too upset, calmed down.

Add a Comment

Login with GitHub to post a comment