? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
21 Nov 2016

Summary of Changes

This PR unblocks several extensions (component/plugins) in installation sql files so they can be disabled in "Extensions" -> "Manage".

Extensions unlocked in this PR:

  • com_wrapper (only used if the user wants too)
  • com_messages (only used if the user wants too)
  • com_redirect (only used if the user wants too)
  • plg_editors_codemirror (an editor - used in template manager)
  • plg_system_languagefilter (only used in multilngual sites)
  • plg_system_cache (only used if the user wants too)
  • plg_system_redirect (only used if the user wants too)
  • plg_quickicon_joomlaupdate (only used if the user wants too - doubt)
  • plg_quickicon_extensionupdate (only used if the user wants too - doubt)
  • plg_quickicon_phpversioncheck (only used if the user wants too)

Testing Instructions

Just code review.

Documentation Changes Required

None.

Note

this PR only does the installation part to validate the concept and the extensions to be unlocked.
if accepted i will add the update SQLs.

avatar andrepereiradasilva andrepereiradasilva - open - 21 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 21 Nov 2016
Status New Pending
avatar mbabker
mbabker - comment - 21 Nov 2016

plg_editors_codemirror (it's an optional editor)

IIRC it's hardcoded into the template manager so I don't think I'd change this one.

avatar andrepereiradasilva andrepereiradasilva - change - 21 Nov 2016
The description was changed
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - edited - 21 Nov 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

IIRC it's hardcoded into the template manager so I don't think I'd change this one.

yes, you are right https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_templates/models/template.php#L368

so i guess that one needs to be locked ... anyway i think by reading the code we get a warning if we have it disabled.

avatar brianteeman
brianteeman - comment - 21 Nov 2016

Yes that's correct

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

ok i will lock that one again

avatar andrepereiradasilva andrepereiradasilva - change - 21 Nov 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - edited - 21 Nov 2016
avatar brianteeman
brianteeman - comment - 21 Nov 2016

I am not convinced about some of this. Some of these are dependant on
others but this enables them to be disabled independently. Pretty sure
thats why they are protected

avatar joomla-cms-bot joomla-cms-bot - change - 21 Nov 2016
Category SQL Installation Postgresql MS SQL
avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

@brianteeman if you are referring to com_redirect and plg_system_redirect, look at com_search and com_finder and their plugins they are all unlocked.

avatar brianteeman
brianteeman - comment - 21 Nov 2016

It's different. Com_redirect requires plg_redirect. Com_search doesn't
require plg_search_contact

On 21 Nov 2016 5:14 p.m., "andrepereiradasilva" notifications@github.com
wrote:

@brianteeman https://github.com/brianteeman if you are referring to
com_redirect and plg_system_redirect, look at com_search and com_finder and
their plugins they are all unlocked.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#12961 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8S3pLiMQcNoqSvEBpepdT5q2s1VKks5rAdFqgaJpZM4K4csv
.

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

well i tried to disable com_redirect and enable plg_system_redirect to check the worst case scenario, and i think the only thing that happens is that i cannot manage the redirect urls, it probably will use the ones already in that are already in the database table.
and that's the worst case scenario, ie, when a user disable the com_redirect and enables the plg_system_redirect plugin.

avatar brianteeman
brianteeman - comment - 21 Nov 2016

And what happens when the user clicks on the link in the warning to enable
the plugin?

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

the same that happens when click on the message in com_finder ... goes to the edit plugin (which is disabled)

avatar ggppdk
ggppdk - comment - 21 Nov 2016

Content plugin "joomla" is using com_messages to send emails about new content submitted in frontend,

I guess the email message will be sent even if component is disabled,
-- but users will not be able to read the messages

I guess administrators will see this and reenable the component ?

Also 3rd party extensions, now assume that the component is always active and never disabled,
i was thinking of using com_messages

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

IIRC com_messages was one of the components that would be removed from the core
Right: https://developer.joomla.org/cms/roadmap.html

assume is not a word i use in programming ?

avatar mbabker
mbabker - comment - 21 Nov 2016

Decoupling was basically cancelled since nobody can figure out how to fully manage it. I think that's the Reader's Digest of that decision anyway.

avatar brianteeman
brianteeman - comment - 21 Nov 2016

the same that happens when click on the message in com_finder ... goes to the edit plugin (which is disabled)

Except that removing the protection enables me to uninstall the plugin
If I do that then I just get a list of plugins when i follow the link

avatar richard67
richard67 - comment - 21 Nov 2016

The problem is that unlocking them enables the user to uninstall them ... and this will cause errors because some of the plugins were once added in the core's schema update scripts ... and this will cause errors on updating from old versions like e.g. 2.5. I think that's why they were locked.


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Nov 2016

hi @richard67 !

if that's so a decision is needed on this, either lock all core extensions (including the ones unlocked now, eg, com_banners, com_contact, com_finder, com_search, etc) or only lock the ones essencial for the cms functioning (like it is now + plus this PR).

avatar richard67
richard67 - comment - 21 Nov 2016

@andrepereiradasilva Yes, I agree. Just wanted to mention that some of the schema update scripts can cause problems if not fully decoupled.

avatar alikon
alikon - comment - 24 Nov 2016

Should'nt this be more correclty handled in the 4.X branch ?

avatar infograf768
infograf768 - comment - 24 Nov 2016

The other problem, as I see it, is that if one uninstalls stuff like the language filter plugin, there is no easy way to install back.
Also, uninstalling them may also delete the related language strings.

avatar brianteeman
brianteeman - comment - 24 Nov 2016

@infograf768 thats true for the existing unprotected extensions

On 24 November 2016 at 07:42, infograf768 notifications@github.com wrote:

The other problem, as I see it, is that if one uninstalls stuff like the
language filter plugin, there is no easy way to install back.
Also, uninstalling them may also delete the related language strings.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#12961 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8eRynjfGoW-_ppFpOJomW5u2Rtzfks5rBT_3gaJpZM4K4csv
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar infograf768
infograf768 - comment - 24 Nov 2016

Alas, yes, Does not mean we have to do it for others until we find a way to get this process more user friendly.

avatar andrepereiradasilva
andrepereiradasilva - comment - 24 Nov 2016

the problem i see is that "lock" does two things:

  • not able to uninstall the extension
  • not able to enable/disable the extension

Maybe it should do just one thing. Not able to uninstall.
And we would have another field for able to enable/disable... Just lanching the idea.

avatar brianteeman
brianteeman - comment - 24 Nov 2016

I hear you about it being useful to disable the extension

On 24 November 2016 at 09:25, andrepereiradasilva notifications@github.com
wrote:

the problem i see is that "lock" does two things:

  • not able to uninstall the extension
  • not able to enable/disable the extension

Maybe it should do just one thing. Not able to uninstall.
And we would have another field for able to enable/disable... Just
lanching the idea.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#12961 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8dOHPH6Whe0XWq_NwbpXidiW40xTks5rBVgKgaJpZM4K4csv
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar andrepereiradasilva
andrepereiradasilva - comment - 27 Nov 2016

please comment on this RFC #13037 if this solves all the issues found here.

avatar andrepereiradasilva andrepereiradasilva - change - 27 Nov 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-11-27 13:10:29
Closed_By andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 27 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - close - 27 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 27 Nov 2016

Add a Comment

Login with GitHub to post a comment