? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
20 Nov 2018

Pull Request for Issue # .

Summary of Changes

This removes remaining code related to column options in com_content. See #18319 and #22840 for details.

Testing Instructions

Code review.

avatar SharkyKZ SharkyKZ - open - 20 Nov 2018
avatar SharkyKZ SharkyKZ - change - 20 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Nov 2018
Category Administration Language & Strings Front End com_content SQL Installation Postgresql Plugins
avatar infograf768
infograf768 - comment - 20 Nov 2018

NOTE

To @wilsonge @laoneo @Bakual @mbabker @rdeutz
Concerning installation/src/Model/LanguagesModel.php, what do we do with the methods which have allowed to install a basic multilingual site at installation time in 3.x as it looks like it has been apparently officially decided to drop this functionality in 4.0 in favor of the multilang sample data plugin.

I still question this decision btw. And it looks like I am not alone on this.

avatar Bakual
Bakual - comment - 20 Nov 2018

Can you ask this question in a separate issue? It's quite offtopic here as this PR is completely unrelated to that question.

avatar infograf768
infograf768 - comment - 20 Nov 2018

I asked it here as the file has been modified and the methods may have to be totally deleted if the decision is confirmed. Just a Note to admins.

avatar Bakual
Bakual - comment - 20 Nov 2018

Still needs an own issue to be properly discussed so it doesn't derail this valid PR here.

avatar infograf768
infograf768 - comment - 20 Nov 2018

sure

avatar SharkyKZ SharkyKZ - change - 20 Nov 2018
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 22 Nov 2018

Personally I would not remove the language strings from the global language file. When I removed all unused strings earlier I left the ones in global files as those are the files recommended for extensions to utilise and I felt it was an unnecessary additional change they would have to make. My 2c

avatar Quy
Quy - comment - 27 Nov 2018

I have tested this item successfully on 62d7cb1


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

avatar Quy Quy - test_item - 27 Nov 2018 - Tested successfully
avatar joomla-cms-bot joomla-cms-bot - change - 1 Mar 2019
Category Administration Language & Strings Front End com_content SQL Installation Postgresql Plugins Unit Tests Repository Administration com_admin SQL Postgresql com_associations com_banners
avatar SharkyKZ SharkyKZ - change - 1 Mar 2019
Labels Added: ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 1 Mar 2019
Category Administration SQL Postgresql Unit Tests Repository com_admin com_associations com_banners Administration Language & Strings Front End com_content SQL Installation Postgresql Plugins
avatar wilsonge
wilsonge - comment - 1 Mar 2019

@zero-24 @SniperSister RIPS please

avatar zero-24 zero-24 - change - 2 Mar 2019
Labels Added: ?
Removed: ?
avatar zero-24
zero-24 - comment - 2 Mar 2019

Fixed rips sorry that it took that long @wilsonge

avatar wilsonge
wilsonge - comment - 2 Mar 2019

@zero-24 doesn't seem to have worked? Or there's a new issue :( No worries about the time

avatar wilsonge wilsonge - change - 21 Mar 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-03-21 16:11:30
Closed_By wilsonge
avatar wilsonge wilsonge - close - 21 Mar 2019
avatar wilsonge wilsonge - merge - 21 Mar 2019
avatar wilsonge
wilsonge - comment - 21 Mar 2019

Thanks!

Add a Comment

Login with GitHub to post a comment