? ? Success

User tests: Successful: Unsuccessful:

avatar dimonzz
dimonzz
5 Aug 2015

Previously com_config was getting help only from local file by reference key.
This PR added possibility to get help remotely by URL from language file, locally by key, remotely by URL provided in component settings and key.
Idea taken from com_categories code.

How to test

1) Install you component
2) Go to config page of you component
3) Try to press "Help" button. You will arrive to Joomla help server
4) Apply the patch
5) Add to lang file constant:
COM_COMPONENTNAME_HELP_URL="https://example/{keyref}.html"
6) Try to press "Help" and you will arrive to page like:
https://example/JHELP_COMPONENTS_COM_COMPONENTNAME_OPTIONS.html
You can use lang file to override JHELP_COMPONENTS_COM_COMPONENTNAME_OPTIONS with something user friendly.
7) Remove COM_COMPONENTNAME_HELP_URL from lang file.
8) Add field "helpURL" to your component config.xml and insert there same url (https://example/{keyref}.html)
9) Try to press "Help" button. Result should be the same as in point 6.
10) Clean helpURL field and save configs.
11) Try to press "Help" button. You should get local help page.

avatar dimonzz dimonzz - open - 5 Aug 2015
avatar dimonzz dimonzz - change - 5 Aug 2015
The description was changed
avatar brianteeman
brianteeman - comment - 5 Aug 2015

Have you seen #6639

On 5 August 2015 at 13:22, dimonzz notifications@github.com wrote:

Previously com_config was getting help only from local file by reference
key.
This PR added possibility to get help remotely by URL from language file,
locally by key, remotely by URL provided in component settings and key.

Idea taken from com_categories code.

You can view, comment on, or merge this pull request online at:

#7632
Commit Summary

  • Added possibility to get help remotely in com_config component view

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#7632.

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

avatar dimonzz dimonzz - change - 5 Aug 2015
The description was changed
avatar dimonzz
dimonzz - comment - 5 Aug 2015

Yes. In this PR also added possibility to use predefined COM_COMPONENT_HELP_URL lang constant. So there is no need to have helpURL field in config.xml

avatar dimonzz dimonzz - change - 5 Aug 2015
The description was changed
avatar joomla-cms-bot joomla-cms-bot - change - 5 Aug 2015
The description was changed
Labels Added: ?
avatar zero-24 zero-24 - change - 5 Aug 2015
Labels Added: ?
avatar mbabker
mbabker - comment - 5 Aug 2015

If I'm not mistaken the helpURL parameter has been supported in Joomla for some time, so this PR just seems like duplicating that logic and not offering anything "better" (subjective measurement).

avatar dimonzz
dimonzz - comment - 5 Aug 2015

helpURL still present and supported. This PR just adds one more way to define it. In my mind, help URL should be predefined by developer, not configured by end user. I think it would be great to add something like <helpServer> tag to manifest in the future.

avatar mbabker
mbabker - comment - 5 Aug 2015

If it's a hidden parameter then the user won't be able to mess with it unless they hack the database.

I'm not trying to shoot down the idea, honestly, but one of my gripes about Joomla right now is there are a lot of ways to accomplish one task, for better or worse. Instead of standardizing on a common behavior (if you want to have a custom help screen set, define <input name="helpURL" type="hidden" /> in your component's config), this adds another way to accomplish the same thing. As a newbie developer it makes me question "which way is right?" and as "the grumpy old guy" it just annoys me when standard behaviors aren't standardized.

avatar dimonzz
dimonzz - comment - 5 Aug 2015

Sounds very reasonable! Glad to reach consensus.

avatar zero-24 zero-24 - change - 5 Aug 2015
Category Administration
avatar zero-24 zero-24 - change - 5 Aug 2015
Status New Pending
avatar roland-d
roland-d - comment - 31 Jul 2016

I am not sure I follow, but this can be closed?


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

avatar dimonzz
dimonzz - comment - 1 Aug 2016

@roland-d i think yes

avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2016
Category Administration Administration Components
avatar zero-24 zero-24 - change - 1 Aug 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-08-01 09:50:26
Closed_By zero-24
avatar zero-24
zero-24 - comment - 1 Aug 2016

Closing as requested


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

avatar zero-24
zero-24 - comment - 1 Aug 2016

Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org/joomla-cms/7632

avatar joomla-cms-bot joomla-cms-bot - close - 1 Aug 2016

Add a Comment

Login with GitHub to post a comment