Bug 19713

Summary: 2 occurrences of OpacShowLibrariesPulldownMobile have not been removed
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: priya.patel
Status: CLOSED FIXED QA Contact: Josef Moravec <josef.moravec>
Severity: trivial    
Priority: P5 - low CC: josef.moravec, priya.patel
Version: unspecifiedKeywords: Academy
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:
Circulation function:
Bug Depends on: 12513    
Bug Blocks:    
Attachments: Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile
Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile
Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile

Description Jonathan Druart 2017-11-29 14:10:34 UTC
t/db_dependent/UsageStats.t:        OpacShowLibrariesPulldownMobile
C4/UsageStats.pm:        OpacShowLibrariesPulldownMobile
Comment 1 priya.patel 2017-12-18 22:49:58 UTC
Created attachment 69870 [details] [review]
Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile

To test:
1/ Grep to find OpacShowLibrariesPullDownMobile
2/ Notice it is in t/db_dependent/UsageStats.t: and C4/UsageStats.pm:
3/ Apply patch
4/ Grep to check OpacShowLibrariesPullDownMobile was removed from 2
files
Comment 2 Katrin Fischer 2017-12-19 06:58:48 UTC
Created attachment 69893 [details] [review]
Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile

To test:
1/ Grep to find OpacShowLibrariesPullDownMobile
2/ Notice it is in t/db_dependent/UsageStats.t: and C4/UsageStats.pm:
3/ Apply patch
4/ Grep to check OpacShowLibrariesPullDownMobile was removed from 2
files

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 3 Katrin Fischer 2017-12-19 06:59:26 UTC
Thx Priya!
Comment 4 Josef Moravec 2017-12-19 09:23:28 UTC
Created attachment 69896 [details] [review]
Bug 19713: Remove 2 occurances of OpacShowLibrariesPullDownMobile

To test:
1/ Grep to find OpacShowLibrariesPullDownMobile
2/ Notice it is in t/db_dependent/UsageStats.t: and C4/UsageStats.pm:
3/ Apply patch
4/ Grep to check OpacShowLibrariesPullDownMobile was removed from 2
files

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 5 Jonathan Druart 2017-12-19 13:38:13 UTC
Priya, you did not specify your lastname in the commit message, is it expected?
Comment 6 Jonathan Druart 2017-12-20 16:44:49 UTC
Patch pushed to master for 18.05, congrats Priya for your first patch in!