Bug 24848

Summary: Help link from label creator batch/layout/template points to card creator in manual
Product: Koha Reporter: Andrew Fuerste-Henry <andrew>
Component: Staff interfaceAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: 1joynelson, gmcharlt, hayleypelham, lucas, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.05, 19.05.10
Circulation function:
Bug Depends on:    
Bug Blocks: 24873    
Attachments: Bug 24848: Improve manual links for the label creator
Bug 24848: Improve manual links for the label creator
Bug 24848: Improve manual links for the label creator
Bug 24848: Improve manual links for the label creator

Description Andrew Fuerste-Henry 2020-03-11 12:17:14 UTC
Clicking the help button from various points within label-manage.pl takes one to the patron card creator area of the manual. It should point to the relevant label bits instead.
Comment 1 Katrin Fischer 2020-03-15 23:54:22 UTC
Created attachment 100714 [details] [review]
Bug 24848: Improve manual links for the label creator

Some of the manual links in the label creator tool were
not working correctly or pointing to the wrong pages.

A problem is the label-manage.pl file. This can show
different data depending on an added parameter. Example:
/cgi-bin/koha/labels/label-manage.pl?label_element=template

But it appears that parameters will be ignored when creating
links to the manual. So this patch links to the beginning
of the label creator section in the manual for now.

To test:
- Go to the lable creator
- Try the various pages of the tool
  - Note: All 'New ...' links should point to the
    correct section in the manual (batches, templates, ...)
    Except: Barcode range. This feature is new and not
    yet documented in the manual. Link goes to label creator section.
  - Note: All 'Manage ...' links should point to the
    beginning of the 'label creator' section in the manual
Comment 2 Katrin Fischer 2020-03-15 23:54:58 UTC
The solution is not perfect - label-manage.pl is a problem still (see patch description).
Comment 3 Katrin Fischer 2020-03-15 23:55:38 UTC
Sorry Andrew for stealing this one - noticed too late. :(
Comment 4 Andrew Fuerste-Henry 2020-03-17 14:17:02 UTC
Not a problem at all! Thanks for the patch!
Comment 5 Andrew Fuerste-Henry 2020-03-17 14:25:10 UTC
Created attachment 100899 [details] [review]
Bug 24848: Improve manual links for the label creator

Some of the manual links in the label creator tool were
not working correctly or pointing to the wrong pages.

A problem is the label-manage.pl file. This can show
different data depending on an added parameter. Example:
/cgi-bin/koha/labels/label-manage.pl?label_element=template

But it appears that parameters will be ignored when creating
links to the manual. So this patch links to the beginning
of the label creator section in the manual for now.

To test:
- Go to the lable creator
- Try the various pages of the tool
  - Note: All 'New ...' links should point to the
    correct section in the manual (batches, templates, ...)
    Except: Barcode range. This feature is new and not
    yet documented in the manual. Link goes to label creator section.
  - Note: All 'Manage ...' links should point to the
    beginning of the 'label creator' section in the manual

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Comment 6 Marcel de Rooy 2020-03-20 13:36:07 UTC
Created attachment 101108 [details] [review]
Bug 24848: Improve manual links for the label creator

Some of the manual links in the label creator tool were
not working correctly or pointing to the wrong pages.

A problem is the label-manage.pl file. This can show
different data depending on an added parameter. Example:
/cgi-bin/koha/labels/label-manage.pl?label_element=template

But it appears that parameters will be ignored when creating
links to the manual. So this patch links to the beginning
of the label creator section in the manual for now.

To test:
- Go to the lable creator
- Try the various pages of the tool
  - Note: All 'New ...' links should point to the
    correct section in the manual (batches, templates, ...)
    Except: Barcode range. This feature is new and not
    yet documented in the manual. Link goes to label creator section.
  - Note: All 'Manage ...' links should point to the
    beginning of the 'label creator' section in the manual

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Kyle M Hall (khall) 2020-03-20 13:41:42 UTC
Created attachment 101110 [details] [review]
Bug 24848: Improve manual links for the label creator

Some of the manual links in the label creator tool were
not working correctly or pointing to the wrong pages.

A problem is the label-manage.pl file. This can show
different data depending on an added parameter. Example:
/cgi-bin/koha/labels/label-manage.pl?label_element=template

But it appears that parameters will be ignored when creating
links to the manual. So this patch links to the beginning
of the label creator section in the manual for now.

To test:
- Go to the lable creator
- Try the various pages of the tool
  - Note: All 'New ...' links should point to the
    correct section in the manual (batches, templates, ...)
    Except: Barcode range. This feature is new and not
    yet documented in the manual. Link goes to label creator section.
  - Note: All 'Manage ...' links should point to the
    beginning of the 'label creator' section in the manual

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Marcel de Rooy 2020-03-20 13:43:38 UTC
Kyle: we're clashing now
Comment 9 Martin Renvoize (ashimema) 2020-03-20 15:43:53 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 10 Joy Nelson 2020-04-03 21:21:45 UTC
backported to 19.11.x for 19.11.05
Comment 11 Lucas Gass (lukeg) 2020-04-13 19:53:25 UTC
backported to 19.05.x for 19.05.10
Comment 12 Hayley Pelham 2020-04-15 23:34:16 UTC
Not an issue in 18.11, not backporting to 18.11.x series.