? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
12 Oct 2016

Looks like the french help site has been updated (at some point) to use https

Changing it here saves a redirect ;)

avatar brianteeman brianteeman - open - 12 Oct 2016
avatar brianteeman brianteeman - change - 12 Oct 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Oct 2016
Category Administration
avatar joomla-cms-bot joomla-cms-bot - change - 12 Oct 2016
Labels Added: ?
avatar mbabker
mbabker - comment - 12 Oct 2016

You can save a second redirect by changing from https://help.joomla.fr/3 to https://help.joomla.fr ?

BTW what's the source for this stuff? https://help.joomla.fr/index.php?option=com_help&keyref=Help36:Start_Here loads a page for 3.2...

avatar brianteeman
brianteeman - comment - 12 Oct 2016

@mbabker removed the second redirect

avatar infograf768
infograf768 - comment - 12 Oct 2016

Any change for this link has also to be done through the VPN as the Refresh button would imho get it back from there.

BTW what's the source for this stuff? https://help.joomla.fr/index.php?option=com_help&keyref=Help36:Start_Here loads a page for 3.2...

Indeed, the French community made its own help site at the time as docs was only providing it in English. It has not been updated since.
In fact, just tested and if one chooses default help and set backend language to French, one still gets English help.

Note: although the French help is displaying fine when using the help button, it does not here when using index.php?option=com_admin&view=help
Here I get an error:
Load denied by X-Frame-Options: https://help.joomla.fr/index.php?option=com_help&keyref=Help36:Components_Banners_Categories_Edit does not permit cross-origin framing.(unknown)

avatar mbabker
mbabker - comment - 12 Oct 2016

I assume you mean https://update.joomla.org/helpsites/helpsites.xml because this controller is the only place in core I could find referencing anything off-site for that (which BTW should be converted to use the JHttp API).

In fact, just tested and if one chooses default help and set backend language to French, one still gets English help.

Core bug? Open your browser's network inspection tab and see what it's requesting; if it isn't appending the language code to the keyref value or adding a lang key then the proxy won't look for a translated page.

Load denied by X-Frame-Options

Get in touch with whomever is running that service, seems they need to tweak their server configuration to let things work correctly.

avatar mbabker
mbabker - comment - 12 Oct 2016

I'll answer the "still seeing English" part for you.

JHelp::createUrl() is what's used to build these URLs, it looks for certain placeholders to replace with values in the URLs (look in the method for the full list). None of the placeholders for language support are there. In fact, there is only one place in core adding the lang variable to the URL and that's AdminModelHelp::getLatestVersionCheck(). So the XML needs to be updated to include the relevant placeholders so that language codes can be included and the backend help gets localized.

And our server will explicitly check for conditions where there isn't a localized page for a language and fall back onto English. See the help screen model from the app's repo (mainly the getPage() method). I've explicitly tested both methods of passing language codes in and the fallback routine is fine (test out https://help.joomla.org/proxy/index.php?keyref=Help36:Start_Here&lang=fr and https://help.joomla.org/proxy/index.php?keyref=Help36:Start_Here/ar as examples).

avatar infograf768
infograf768 - comment - 12 Oct 2016

@mbabker
I was using VPN.joomla.org at the time and a specific update FTP to place this (No need to give more details here). I honestly don't remember well where.

Concerning the help loaded when in backend using fr-FR lang and default core joomla help, I do not get the lang variable at all.
screen shot 2016-10-12 at 18 53 53

avatar mbabker
mbabker - comment - 12 Oct 2016

We've got that XML file at https://github.com/joomla/update.joomla.org/blob/master/www/helpsites/helpsites.xml so whenever this PR is merged someone can make a PR there and it'll get deployed up.

If we want to address the untranslated help, the English site can be changed to this definition and it should work:

<site tag="en-GB" url="https://help.joomla.org/proxy?keyref=Help{major}{minor}:{keyref}&amp;lang={langcode}">English (GB) - Joomla help wiki</site>
avatar infograf768
infograf768 - comment - 12 Oct 2016

from ipad
if thedefault help is corrected, no use to prpose anymore the custom french one imho
will let this topic known to the french community asap

avatar infograf768
infograf768 - comment - 13 Oct 2016

@mbabker
Although using <site tag="en-GB" url="https://help.joomla.org/proxy?keyref=Help{major}{minor}:{keyref}&amp;lang={langcode}">English (GB) - Joomla help wiki</site> does help get some stuff in French, there is not much there yet.

It is nevertheless a bug and necessary improvement. It should be solved both in core and in https://github.com/joomla/update.joomla.org/blob/master/www/helpsites/helpsites.xml

To test one has to enter the url in configuration.php:
public $helpurl = 'https://help.joomla.org/proxy?keyref=Help{major}{minor}:{keyref}&amp;lang={langcode}';

Therefore I think the French help is still necessary for the moment although some of it is obsolete.

Note: some of the English help is also obsolete.

avatar infograf768
infograf768 - comment - 13 Oct 2016

I made a PR towards update.joomla.org.
@Brianteeman
Can you add here the changes to get <site tag="en-GB" url="https://help.joomla.org/proxy?keyref=Help{major}{minor}:{keyref}&amp;lang={langcode}">English (GB) - Joomla help wiki</site>

And then we can merge this in 3.6.3

avatar brianteeman
brianteeman - comment - 13 Oct 2016
avatar infograf768
infograf768 - comment - 13 Oct 2016

Looks like test have to be updated to please Travis
1) JFormFieldHelpsiteTest::testGetInput
etc.

avatar infograf768
infograf768 - comment - 13 Oct 2016

@brianteeman
Looks like a very simple change to please travis

avatar brianteeman
brianteeman - comment - 13 Oct 2016

I am not touching tests as I have zero knowledge of how that works. So if
someone can please do a PR against my branch to fix that then I will merge
it

On 13 October 2016 at 10:03, infograf768 notifications@github.com wrote:

@brianteeman https://github.com/brianteeman
Looks like a very simple change to please travis


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

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

avatar infograf768
infograf768 - comment - 13 Oct 2016

here it is
brianteeman#12

avatar brianteeman
brianteeman - comment - 13 Oct 2016

@infograf768 merged it - lets see

avatar infograf768 infograf768 - change - 13 Oct 2016
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 13 Oct 2016

For 3.6.3

RTC

@rdeutz


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

avatar joomla-cms-bot joomla-cms-bot - change - 13 Oct 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 14 Oct 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-10-14 10:08:18
Closed_By rdeutz
avatar rdeutz rdeutz - close - 14 Oct 2016
avatar rdeutz rdeutz - merge - 14 Oct 2016
avatar joomla-cms-bot joomla-cms-bot - close - 14 Oct 2016
avatar brianteeman
brianteeman - comment - 14 Oct 2016

Thanks

avatar joomla-cms-bot joomla-cms-bot - change - 14 Oct 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment