---- Reported by nengard@gmail.com 2008-07-09 09:00:18 ---- When on Global System Preference the help button should correspond to the tab you're on instead of to a complete list of preferences regardless of the tab you're on. PER CHAT: <nengard> i'm going to assume the answer is no - but i want to ask just in case - is there any way to have the help link on the system preferences page different for each tab on the page? right now it's all system preferences - but it's not really all system preferences - so it's silly <atz> nengard: yeah, that part is crappy. it should be possible to do something extra there <nengard> atz should i submit as enhancement or is that a quick fix we could do so i can update the help to match my documentation? <atz> enhancement --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:49 UTC --- This bug was previously known as _bug_ 2326 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2326 Actual time not defined. Setting to 0.0 CC member bchurch@ptfs.com does not have an account here CC member jmf@liblime.com does not have an account here CC member joe.atzberger@liblime.com does not have an account here
*** Bug 2886 has been marked as a duplicate of this bug. ***
Digging this up again, it seems like it would be simple to implement from the script side of things. We'd have to create a special case for admin/preferences where the query string was processed. After that, create separate preferences help files based on a simple pattern like "help/admin/preferences_staff_client.tt" Is isolating the query string a matter of a good regular expression, or might there be some existing URL-crunching code to use?
Created attachment 60166 [details] [review] Bug 2326: Individual help pages for every tab in system preferences To test: 1) Apply patch 2) Confirm that when not on any particular tab, the general help shows as usual 3) Confirm that when on a specific tab, the help that shows up is correct 4) Confirm that all links in the help file work correctly 5) Confirm this works for every tab Sponsored-by: Catalyst IT
I went through all tabs, works OK. However I did not read everything. Found one small typo in a title: "Patorns System Preferences"
Created attachment 60482 [details] [review] Bug 2326: Individual help pages for every tab in system preferences To test: 1) Apply patch 2) Confirm that when not on any particular tab, the general help shows as usual 3) Confirm that when on a specific tab, the help that shows up is correct 4) Confirm that all links in the help file work correctly 5) Confirm this works for every tab Sponsored-by: Catalyst IT
Re-tested. QA tools found 2 issues (sorry, I was not aware of with my previous test): ----------------------- FAIL koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/preferences_circulation.tt OK forbidden patterns OK git manipulation FAIL spelling adminstration ==> administration everytime ==> every time OK tt_valid OK valid_template -------------------------- FAIL koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/preferences_opac.tt FAIL forbidden patterns forbidden pattern: opac-tmpl should certainly replaced with [% interface %] (line 376) forbidden pattern: opac-tmpl should certainly replaced with [% interface %] (line 110)
Created attachment 60537 [details] [review] Bug 2326: Individual help pages for every tab in system preferences To test: 1) Apply patch 2) Confirm that when not on any particular tab, the general help shows as usual 3) Confirm that when on a specific tab, the help that shows up is correct 4) Confirm that all links in the help file work correctly 5) Confirm this works for every tab Sponsored-by: Catalyst IT
(In reply to Marc Véron from comment #6) > Re-tested. QA tools found 2 issues (sorry, I was not aware of with my > previous test): > ----------------------- > > FAIL > koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/preferences_circulation.tt > OK forbidden patterns > OK git manipulation > FAIL spelling > adminstration ==> administration > everytime ==> every time > OK tt_valid > OK valid_template > > > -------------------------- I have fixed these in latest patch. > > FAIL koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/preferences_opac.tt > FAIL forbidden patterns > forbidden pattern: opac-tmpl should certainly replaced with [% interface > %] (line 376) > forbidden pattern: opac-tmpl should certainly replaced with [% interface > %] (line 110) I did not fix these because the instances of 'opac-tmpl' in the patch are examples of paths to css files, not actual URLs. Anyway, if i used [% interface %] in their place, it would change to intranet-tmpl and incorrect.
Created attachment 60540 [details] [review] Bug 2326: Individual help pages for every tab in system preferences To test: 1) Apply patch 2) Confirm that when not on any particular tab, the general help shows as usual 3) Confirm that when on a specific tab, the help that shows up is correct 4) Confirm that all links in the help file work correctly 5) Confirm this works for every tab Sponsored-by: Catalyst IT Followed testplan, works as expected. For QA tools not passing with "forbidden pattern: opac-tmpl should certainly replaced with [% interface %]" for file preferences_opac.tt see comment #8 Signed-off-by: Marc Véron <veron@veron.ch>
I have the feeling that with this patch we are introducing another hard-to-keep-updated thing. I am wondering if a link to the online manual would not be better/enough. Otherwise we need a script + tests to make sure the help files are updated. For instance the patch is already wrong, it lists 'Persona' whereas it has been removed.
This bug predates the switch to using the online manual for help, but it makes me wonder if we could do more to link people to the relevant section of the online manual.
Martin accidentally fixed this one on bug 31465. Patch here would no longer work, as we have removed the help files since in favor of linking to the manual. *** This bug has been marked as a duplicate of bug 31465 ***
Excellent [U+1F60A]