Bug 14077

Summary: Remove unused AUTH_UPDATE_FAILED from moremember.tt
Product: Koha Reporter: Marc Véron <veron>
Component: PatronsAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: chris, gmcharlt, kyle.m.hall, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13910
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: 13910    
Bug Blocks:    
Attachments: Bug 14077 - Remove unused AUTH_UPDATE_FAILED from moremember.tt
Bug 14077 - Remove unused AUTH_UPDATE_FAILED from moremember.tt
[PASSED QA] Bug 14077 - Remove unused AUTH_UPDATE_FAILED from moremember.tt

Description Marc Véron 2015-04-29 09:30:43 UTC
'AUTH_UPDATE_FAILED' in moremember.tt is never set and should be removed (see comment #6 on Bug 13910).

To verify: git grep 'AUTH_UPDATE_FAILED'
Result: Only one occurence (in moremember.tt)
Comment 1 Marc Véron 2015-04-29 09:40:07 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2015-04-29 10:30:25 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2015-04-29 20:11:07 UTC
Created attachment 38685 [details] [review]
[PASSED QA] Bug 14077 - Remove unused AUTH_UPDATE_FAILED from moremember.tt

'AUTH_UPDATE_FAILED' in moremember.tt is never set and should be removed (see comment #6 on Bug 13910).

To verify: git grep 'AUTH_UPDATE_FAILED'
Result: Only one occurence (in moremember.tt)

Test plan:
- Apply patch (on top of Bug 13910)
- Verify that the part with AUTH_UPDATE_FAILED is removed from moremember.tt
- Go to a patron's detail page and verify that nothing is broken (Home > Patrons > Patrons details for...)

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This has been introduced by
  commit 1a91801b06607ee9e5f8c21d531a7b7a3ca3b683
  Date:   Sat Dec 1 20:56:27 2007 -0600
      Add Staff member type. Add permissions checks for bug# 1269
      and related permissions on memberentry and mods.
But was already not used...

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Tomás Cohen Arazi (tcohen) 2015-05-14 14:57:06 UTC
Patch pushed to master.

Thanks Marc!
Comment 5 Chris Cormack 2015-05-17 23:01:39 UTC
Pushed to 3.18.x will be in 3.18.7