Bug 14439

Summary: Typo in Bug 14408 regexp
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: Staff interfaceAssignee: Fridolin Somers <fridolin.somers>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: chris, gmcharlt, liz, tomascohen
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:
Bug Depends on: 14408    
Bug Blocks:    
Attachments: Bug 14439 - Typo in Bug 14408 regexp
[SIGNED OFF] Bug 14439: Typo in Bug 14408 regexp
Bug 14439 - Typo in Bug 14408 regexp
Bug 14439: Add test - template path should finished by .tt
Bug 14439: Add test - template path should finish by .tt
Bug 14439: Typo in Bug 14408 regexp
Bug 14439: Add test - template path should finish by .tt

Description Fridolin Somers 2015-06-23 13:43:51 UTC
In Bug 14408 first patch, the regexp used needs an escape on dot and does not need an ending "?"
Comment 1 Fridolin Somers 2015-06-23 13:59:53 UTC Comment hidden (obsolete)
Comment 2 Tomás Cohen Arazi 2015-06-23 14:04:20 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-06-23 14:10:58 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2015-06-23 14:11:55 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2015-06-23 14:16:48 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2015-06-23 14:20:36 UTC
Created attachment 40531 [details] [review]
Bug 14439: Typo in Bug 14408 regexp

In Bug 14408 first patch, the regexp used needs an escape on dot and does not need an ending "?"

Test plan :
  - prove t/db_dependent/Auth.t

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 7 Jonathan Druart 2015-06-23 14:20:41 UTC
Created attachment 40532 [details] [review]
Bug 14439: Add test - template path should finish by .tt

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 8 Liz Rea 2015-06-23 22:24:26 UTC
I've added the fix, but not the tests, to 3.18.
Comment 9 Tomás Cohen Arazi 2015-06-24 14:34:16 UTC
Patch pushed to master.

Thanks Frido!
Comment 10 Fridolin Somers 2015-06-26 15:53:04 UTC
regexp is an old friend of mine ;)
Comment 11 Chris Cormack 2015-07-13 21:13:14 UTC
Pushed to 3.20.x will be in 3.20.2