Bug 20658 - Move template JavaScript to the footer: Installer and onboarding
Summary: Move template JavaScript to the footer: Installer and onboarding
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Target Milestone: ---
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Whiteboard:
Keywords:
: 22309 (view as bug list)
Depends on:
Blocks: 17858
  Show dependency treegraph
 
Reported: 2018-04-25 16:23 UTC by Owen Leonard
Modified: 2019-02-11 16:56 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments
Bug 20658: Move template JavaScript to the footer: Installer and onboarding (23.23 KB, patch)
2018-04-25 17:50 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 20658: Move template JavaScript to the footer: Installer and onboarding (23.71 KB, patch)
2018-09-15 22:38 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 20658: Move template JavaScript to the footer: Installer and onboarding (23.77 KB, patch)
2018-09-15 22:54 UTC, Claire Gravely
Details | Diff | Splinter Review
Bug 20658: (follow-up) Fix for QA: Add missing html filters (3.12 KB, patch)
2018-09-24 12:16 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 20658: (follow-up) Fix for QA: Add missing html filters (3.13 KB, patch)
2018-10-29 14:13 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2018-04-25 16:23:10 UTC
In order to conform to best practices for performance staff client template javascript should be in the footer instead of the header.

This patch will do so for the installer and onboarding templates.
Comment 1 Owen Leonard 2018-04-25 17:50:39 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2018-09-15 22:38:13 UTC Comment hidden (obsolete)
Comment 3 Claire Gravely 2018-09-15 22:54:55 UTC
Created attachment 78912 [details] [review]
Bug 20658: Move template JavaScript to the footer: Installer and onboarding

This patch updates the installer and onboarding templates so that
JavaScript is included in the footer instead of the header. A new
include file is added, installer-intranet-bottom.inc, so that only
required JS is being loaded.

Some markup changes are included to improve validity.

To test, apply the patch and start with a blank Koha database. Run
through the installer and onboarding processes and confirm that there
are no problems. Check especially JS-related interactions like
datepickers and fields which are validated via JS, like the password
inputs.

NOTE: The qa tool will flag this patch for not using the Asset plugin,
but it doesn't work correctly for installer files.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
Comment 4 Jonathan Druart 2018-09-22 13:48:06 UTC
html filters have been removed in the include file.
Comment 5 Owen Leonard 2018-09-24 12:16:56 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2018-10-27 20:53:10 UTC
QA script is quite unhappy, can you please check?

OK	koha-tmpl/intranet-tmpl/prog/en/includes/installer-doc-head-close.inc
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   OK	  valid_template

 FAIL	koha-tmpl/intranet-tmpl/prog/en/includes/installer-intranet-bottom.inc
   OK	  filters
   FAIL	  forbidden patterns
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 4)
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 1)
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 2)
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 3)
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 5)
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/installer/auth.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/installer/step1.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/installer/step2.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/installer/step3.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep1.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep2.tt
   OK	  filters
   FAIL	  forbidden patterns
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 132)
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep3.tt
   OK	  filters
   FAIL	  forbidden patterns
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 116)
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep4.tt
   OK	  filters
   FAIL	  forbidden patterns
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 52)
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep5.tt
   OK	  filters
   FAIL	  forbidden patterns
		forbidden pattern: Do *not* include full path to js files, use the Asset TT plugin (bug 20538) (line 139)
   OK	  git manipulation
   OK	  spelling
   OK	  tt_valid
   FAIL	  valid_template
		: filter not found
Comment 7 Owen Leonard 2018-10-29 14:13:15 UTC
Created attachment 81504 [details] [review]
Bug 20658: (follow-up) Fix for QA: Add missing html filters

This patch adds missing filters to the installer header and footer
include files.
Comment 8 Owen Leonard 2019-02-11 16:56:40 UTC
*** Bug 22309 has been marked as a duplicate of this bug. ***