This comes after a chat I had with @infograf768 on Slack:
The proposal is to modify the descriptions of the different updates options.
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_CUSTOM="You are connected to a %s - This is not an official Joomla! update channel."
; @deprecated for 4.0 replace COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_LTS with COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_CURRENT
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_LTS="You are connected to the %s update channel - This channel is designed for the current major release and provides all minor and patch updates of your version until the support for your version ends."
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_NOCHANGE="You are not connected to an update channel and will not recieve updates. This setting will compromise the security of your site."
; @deprecated for 4.0 replace COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_STS with COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_ALL
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_STS="You are connected to the %s update channel - This channel is designed for major releases and provides all major, minor and patch updates. So you are always on the last version of Joomla!."
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_TESTING="You have chosen the %s update channel - This channel is designed for testing new releases and issues in Joomla!.
It is only intended for JBS (Joomla! Bug Squad) members and others within the Joomla! community who are testing upcoming releases. DO NOT USE THIS SETTING ON A PRODUCTION SITE."
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_CURRENT="You have chosen the %s update channel - Through this channel you'll receive notfications for updates of the current major release (3.x) but you will NOT be notified of a possible new major releases (4.x)"
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_ALL="You have chosen the %s update channel - Through this channel you'll receive notfications for updates of the current major release (3.x) and you will also be notified of a possible new major releases (4.x)"
If there is a consensus, I'll turn this into a PR (possibly together with other related strings). Suggestions and criticism are welcome (English is not my mother-tongue...)
BTW for B/C reasons we need to stay with _STS and _LTS anyway until 4.x is released.
Suggestions and criticism are welcome
maybe ā%sā so the "Update channel name" is clear isolated.
@zero-24 Talking with infograf768 I had the impression that he was open to new names for the constants, but I'm not opposing at all if this is for whatever reason a problem.
I also had the impression that the 2 new strings were meant to be used right now, with J! 3.4.0, but I prefer to leave him the word...
agreed for the quotes!
as for the constant names, _MAJOR can give the impression that it is related to new major releases only
... of course there is a typo notfications notifications
maybe, to be even more explicit:
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_ALL="You have chosen the %s update channel - Through this channel you'll receive notifications for updates of the current major release (3.x) and you will also be notified of a possible new major releases (4.x) whose compatibility with your environment must be thoroughly assessed"
@smanzi if we use ALL and/or CURRENT we need to fix also here: https://github.com/joomla/joomla-cms/pull/3681/files
as for the constant names, _MAJOR can give the impression that it is related to new major releases only
This constand is never shown to the user.
@smanzi if we use ALL and/or CURRENT we need to fix also here: https://github.com/joomla/joomla-cms/pull/3681/files
and need to use / handle LTS/STS for B/C reasons until 4.x here: https://github.com/joomla/joomla-cms/pull/3917/files
I wrote new string at @infograf768 request last week. Don't know what he
did with them
On 3 Dec 2014 20:52, "zero-24" notifications@github.com wrote:
@smanzi https://github.com/smanzi if we use ALL and/or CURRENT we need
to fix also here: https://github.com/joomla/joomla-cms/pull/3681/filesand need to use / handle LTS/STS for B/C reasons until 4.x here:
https://github.com/joomla/joomla-cms/pull/3917/filesā
Reply to this email directly or view it on GitHub
#5308 (comment).
oh sorry @brianteeman than i have missunderstand you about this. Thanks.
... we seems to be a bit out of sync...
This is what I gave to @infograf768 at his request last week and he said he was doing a PR
" You are connected to the 3.x update stream - This stream is designed for major releases and provides all major, minor and patch updates. So you are always on the current version of Joomla! 3.x.
" This stream reserved for future use. - You are connected to the 4.x update stream - This stream is designed for future major release that may be backwards incompatible and is provided to enable you to update to the next major version of Joomla. Once you have updated your site to 4.x this stream provides all major, minor and patch updates. So you are always on the latest version of Joomla! 4.x."
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5308.
It is OK to use new constants in 3.4.0. Therefore if a lang pack has not been updated, the user will at least get the information in English. As anyway we drop 3.2.x, it is really better.
@brianteeman
The point is, if I understand well, that the second choice will also propose the 3.x update, which makes it confusing. Therefore why I could not make up my mind.
It is OK to use new constants in 3.4.0.
Sure but we need to take care of this: https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/views/default/tmpl/default.php#L14-16
So until 4.x we need to use LTS and STS anyway
As that the values we have here: https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/config.xml#L16-18
If I understand well, we can change "COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_" to anyhting we like and is new, can't we?
If I understand well, we can change "COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_" to anyhting we like and is new, can't we?
hmm not so easy. Let me explain.
Current Joomla Installation has a Setup e.g. with the setting lts
/ sts
This result into the Language String (see: https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/views/default/tmpl/default.php#L14-16):
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_LTS
or
COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_STS
If we add a language string for All/ Current or Major/Minor we can't use it as we have here:
https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/config.xml#L16-18
Only lts
and sts
and IIRC we can't Change this until 4.x for B/C
So sure we can add the strings but we never us it with the current code. (That is the reason why the *LTS and *STS strings are the same as the *Major and *Minor strings)
If we want to use it we need here: https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/views/default/tmpl/default.php#L14-16
Somthing like:
$updateOption = strtoupper($params->get('updatesource', 'lts'));
if ($updateOption == 'LTS')
{
$updateOption = 'CURRENT';
}
elseif ($updateOption == 'STS')
{
$updateOption = 'ALL';
}
$langKey = 'COM_JOOMLAUPDATE_VIEW_DEFAULT_UPDATES_INFO_' . $updateOption;
$updateSourceKey = JText::_('COM_JOOMLAUPDATE_CONFIG_UPDATESOURCE_' . $updateOption);
This would also solove the current issue with
You are connected to the **Long Term Support (recommended)** update stream - This stream is designed for the current major release and provides all minor and patch updates of your version until the support for your version ends.
And convert them to
You are connected to the **Current Minor and patch Release** update stream - This stream is designed for the current major release and provides all minor and patch updates of your version until the support for your version ends.
Hello everybody!
I've looked at the code and now understand what the fuss is about: it all comes from the fact that we are building the language keys by a string concatenation of a base string and $updateOption.
As elegant as this can be, I think that by simply substituting that code with a switch($updateOption) {}
block will solve this issue.
As far as regards the parameter values 'lts' and 'sts' there is not much we can do as they are stored as is (i.e. strings) and not numeric values as I expected. Unless we want to introduce some code to update the configuration we must live with those and call it a day... No big deal: only coders will see those and we can add a comment clarifying what their mean is in today context.
On the contrary I think that the language constant names have far more relevance as not only coders but translators too will be exposed to those names. For this reason I am strongly against the usage of terms containing "_MINOR" and "_MAJOR": a lesser skilled translator can misunderstand their significance and think that "_MINOR" means "will be updated to every version, even the minor ones (and of course the major ones!)" and "_MAJOR" means "will be updated only when a major release is available (and to the hell the minor ones!)" I think this can be an easy misunderstanding... we don't have to think with a "coder head" in those circumstances: translators (and end users) may don't know about semantic versioning!
One more (important) thing that I really don't understand is the 'nochange' value, whose description is:
COM_JOOMLAUPDATE_CONFIG_UPDATESOURCE_NOCHANGE="Currently configured (no change)"
if you choose that option, then you are told that: "You are not connected to an update stream and will not recieve updates. This setting will compromise the security of your site." I'm pretty sure there is something wrong with that!!
JM, may I give a try to fix all of the above and push a PR?
One more consideration: does it really makes any sense to propose now the option of an automatic update to a version which so far away and if automatically applied will likely send a site belly up and maybe find us in court defending in a civil cause (we will probably win, but anyway...)?
Please, also have a look at this (it comes from a message posted in joomla-dev-cms Google Group):
Under the update component. I've set the update source to "LTS". So I guess the source should offe 3.x then and not 2.5. This should be changed in 2.5.
I have to select STS to update to 3.x
-Vegard
About the "nochange" option, it seems that this is really broken and with a broken logic behind it:
Therefore, I propose to equate that "nochange" option to the default update channel (safer choice).
I'm preparing a PR... please hold on...
Status | New | ⇒ | Closed |
Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org/joomla-cms/5308
closing since we have a PR here: #5324
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5308.
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-12-04 21:54:22 |
Labels |
Added:
?
|
thanks @smanzi but we cant change the language constants to _ALL and _CURRENT. What is the problem with _Major and _minor? (as it say the same :D)
About the english i hope @brianteeman can have a look into it.