[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Julian Maurice  changed:

   What|Removed |Added

 Status|Pushed to Master|Pushed to Stable

--- Comment #67 from Julian Maurice  ---
Follow-up pushed to 3.22.x, will be in 3.22.7

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #66 from Nick Clemens  ---
*** Bug 16479 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall  changed:

   What|Removed |Added

 Status|Passed QA   |Pushed to Master

--- Comment #65 from Kyle M Hall  ---
Thanks Marcel! Follow-up has been pushed to master for 16.05.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #51421|0   |1
is obsolete||

--- Comment #64 from Marcel de Rooy  ---
Created attachment 51422
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51422=edit
Bug 13871: [QA Follow-up] Fix small regression on AF with branchcode in Patron
Info request

If you enable send_patron_home_library_in_af in your sip account, you
want a separate AF field for the home branch.

Test plan:
Send a 63 (Patron Info) and verify that you have an extra AF.

Signed-off-by: Marcel de Rooy 
Tested Patron Status and Patron Info.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #63 from Katrin Fischer  ---
I agree - fixing it is better.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #62 from Marcel de Rooy  ---
@RM: Handled as QA follow-up. Please check last patch and push that one only.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Pushed to Stable|ASSIGNED

--- Comment #61 from Marcel de Rooy  ---
(In reply to Katrin Fischer from comment #59)
> The problem I see here is that it's already in a stable release. So a revert
> is  not as easy as just getting it out of master.

It is a trivial fix. So why talk revert?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Status|ASSIGNED|Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #60 from Marcel de Rooy  ---
Created attachment 51421
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51421=edit
Bug 13871: [QA Follow-up] Fix small regression on AF with branchcode in Patron
Info request

If you enable send_patron_home_library_in_af in your sip account, you
want a separate AF field for the home branch.

Test plan:
Send a 63 (Patron Info) and verify that you have an extra AF.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #59 from Katrin Fischer  ---
The problem I see here is that it's already in a stable release. So a revert is
 not as easy as just getting it out of master.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #58 from Marcel de Rooy  ---
(In reply to Kyle M Hall from comment #56)
> This is
> definitely a regression and needs to be fixed or we may have to revert this
> bug in master before the release.

Note that I have seen more serious regressions than this where the word
"revert" is not used rightaway. Just saying.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #57 from Marcel de Rooy  ---
(In reply to Kyle M Hall from comment #56)
> Yes, SIP2 can have multiple AF fields. 
In that case that is the best route.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #56 from Kyle M Hall  ---
> Hi Nick,
> Your problem should not be that hard to resolve.
> I think we should not rely on the fact that AF only includes the patron
> branch code. It is just a screen message field. And I am not sure if SIP2
> officially supports multiple AF fields btw.
> Could you use field AO that only contains the institution id? But not usable
> for another branch.
> Or - as workaround - add a prefix to the AF home branch part and capture
> only that part with a simple regex or so?

Yes, SIP2 can have multiple AF fields. The branchcode in AF field feature was
requested specifically by Overdrive to be able to track use by home library.
The AO does not transmit the patron home branch, it transmits the branch of the
user used to authenticate as for using SIP. It is not practical or correct to
send the patron branch on the AO field. This is definitely a regression and
needs to be fixed or we may have to revert this bug in master before the
release.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #55 from Marcel de Rooy  ---
(In reply to Nick Clemens from comment #52)
> This patch has altered the way that the patron branch is sent.  Previously
> it was in a separate AF field, now the branch is concatenated onto the other
> messages with a ' -- '
> 
> I am not sure if this is intentional but it is causing problems for one of
> our libraries where this field was captured for statistical purposes. Can we
> adjust this?

Hi Nick,
Your problem should not be that hard to resolve.
I think we should not rely on the fact that AF only includes the patron branch
code. It is just a screen message field. And I am not sure if SIP2 officially
supports multiple AF fields btw.
Could you use field AO that only contains the institution id? But not usable
for another branch.
Or - as workaround - add a prefix to the AF home branch part and capture only
that part with a simple regex or so?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #54 from Mason James  ---
(In reply to Nick Clemens from comment #52)
> This patch has altered the way that the patron branch is sent.  Previously
> it was in a separate AF field, now the branch is concatenated onto the other
> messages with a ' -- '
> 
> I am not sure if this is intentional but it is causing problems for one of
> our libraries where this field was captured for statistical purposes. Can we
> adjust this?

hi Nick, no, it's not an intentional change, and can be changed back

(i think the AF change may have happened in a follow-up patch?)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Nick Clemens  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=16479

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #53 from Katrin Fischer  ---
Hi Nick, can you please open a new bug linked to this one?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Nick Clemens  changed:

   What|Removed |Added

 CC||n...@bywatersolutions.com

--- Comment #52 from Nick Clemens  ---
This patch has altered the way that the patron branch is sent.  Previously it
was in a separate AF field, now the branch is concatenated onto the other
messages with a ' -- '

I am not sure if this is intentional but it is causing problems for one of our
libraries where this field was captured for statistical purposes. Can we adjust
this?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Julian Maurice  changed:

   What|Removed |Added

 Status|Pushed to Master|Pushed to Stable
 CC||julian.maur...@biblibre.com

--- Comment #51 from Julian Maurice  ---
Patches pushed to 3.22.x, will be in 3.22.6

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Brendan Gallagher  changed:

   What|Removed |Added

 Status|Passed QA   |Pushed to Master
 CC||bren...@bywatersolutions.co
   ||m

--- Comment #50 from Brendan Gallagher  ---
Pushed to Master - Should be in the May 2016 release. Thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #49 from Marcel de Rooy  ---
Created attachment 48672
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48672=edit
Bug 13871: [QA Follow-up] Add $server for FID_SCREEN_MSG

When you are ready, you still see that small detail.

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

   Patch complexity|--- |Small patch
 QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl
   |y.org   |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

--- Comment #48 from Marcel de Rooy  ---
QA Comment:
Since this patch waited much too long, I have been working somewhat more on it
to get it thru QA.
My main concerns were:
[1] Change of behavior when sending correct card number but invalid password.
We should get patron info with BL=Y and CQ=N.
Your patch changed the if-structure and handled this situation similar to wrong
card number. So no patron information. 
Handled in a followup.
[2] Adding a SIPconfig attribute like overdrivemode or msg_invalid_credentials
is not really needed. If we can prevent adding such "prefs", the code will be
simpler.
Although the response contains BL and CQ to inform about the checked
credentials, we could just send "Invalid card number" if there is no card (add
AF here).
If the card is OK and the pw is not, we could add Invalid pw to the existing
AF. This does not harm anyone.
Handled in a followup.
[3] We should have a look at Patron Info too. Added a followup.
Added a new report for additional unit tests for changes in Patron
Status/Patron Info.

Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #46 from Marcel de Rooy  ---
Created attachment 48670
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48670=edit
Bug 13871: [QA Follow-up] Adjust Patron Status Request

Conform QA comment on Bugzilla, we do this:

[1] Attribute for overdrive mode/invalid credentials is not really needed.
We can always pass a screen message that card or password is invalid.
[2] If the cardnumber is correct and the password is wrong, we should
still honour the request. The bad password is recognized by BLN and
an additional message in AF.

Signed-off-by: Marcel de Rooy 
Verified by telnetting SIP server.
And tested additionally with the new unit test of bug 15956.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #45 from Marcel de Rooy  ---
Created attachment 48669
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48669=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-



-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

Signed-off-by: Kyle M Hall 
Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #47 from Marcel de Rooy  ---
Created attachment 48671
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48671=edit
Bug 13871: [QA Follow-up] Adjust Patron Info Request

Conform QA comment, Patron Info request is slightly adjusted to be
consistent with changes to Patron Status request.

If the cardnumber is ok and the password is wrong, BL=N is reported but
also add 'Invalid password' in AF.
Additionally, an invalid card number is reported in AF.

Test plan:
[1] Send patron info request for invalid card.
[2] Idem for valid card, no password.
[3] Idem for valid card, good password.
[4] Idem for valid card, wrong password.

Signed-off-by: Marcel de Rooy 
Verified by telnetting SIP server.
And tested additionally with the new unit test of bug 15956.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #44 from Marcel de Rooy  ---
Created attachment 48668
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48668=edit
Bug 13871: [TITLE_AMENDED] Additional changes

The original perltidy patch from Mason has been amended.

The perltidy itself has been moved to a separate patch with the current
perltidyrc applied.

As noted on Bugzilla, the original perltidy patch included some extra
changes:
[1] You prefix timestamp with Sip
This is not actually needed (it is imported), but if we should prefix it,
we should prefix now with C4::SIP::Sip. But you only changed two
occurrences (out of 26). So I remove these two changes.
[2] You remove the $server parameter from two calls of maybe_add:
A closer look at the remaining code tells me that $server is always
passed to maybe_add for FID_SCREEN_MSG. So this only left me the
current whitespace change.
But at least we documented what we did or did not, and why..

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #48386|0   |1
is obsolete||
  Attachment #48387|0   |1
is obsolete||
  Attachment #48392|0   |1
is obsolete||
  Attachment #48393|0   |1
is obsolete||

--- Comment #43 from Marcel de Rooy  ---
Created attachment 48667
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48667=edit
Bug 13871: Adjusted perl tidy on MsgType.pm

Run perltidy pro=xt/perltidyrc on the file.

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=15956

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 Blocks||15956


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15956
[Bug 15956] Rearranging some SIP unit tests (test without SIP server)
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #42 from Marcel de Rooy  ---
(In reply to Marcel de Rooy from comment #41)
> Will continue QA session here later.

Submitting some follow-ups to get this thru QA later this week.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #41 from Marcel de Rooy  ---
Will continue QA session here later.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #40 from Marcel de Rooy  ---
And some advanced wrestling with utf8 characters in the commit message (and the
git version on Debian 7)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #48391|0   |1
is obsolete||

--- Comment #39 from Marcel de Rooy  ---
Created attachment 48393
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48393=edit
Bug 13871 [QA Followup] - Rename parameter to send_msg_on_invalid_credentials

There are likely other services that would use this same feature.
Let's name it something a bit more general.

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #48390|0   |1
is obsolete||

--- Comment #38 from Marcel de Rooy  ---
Created attachment 48392
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48392=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-



-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #48388|0   |1
is obsolete||

--- Comment #36 from Marcel de Rooy  ---
Created attachment 48390
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48390=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-



-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha…'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #48389|0   |1
is obsolete||

--- Comment #37 from Marcel de Rooy  ---
Created attachment 48391
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48391=edit
Bug 13871 [QA Followup] - Rename parameter to send_msg_on_invalid_credentials

There are likely other services that would use this same feature.
Let's name it something a bit more general.

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #35 from Marcel de Rooy  ---
I rebased this patch set on the new perltidy.
The second patch includes comments on the changes from Mason.
This effectively obsoletes the last follow-up from Kyle to fix the namespace;
the namespace is not needed.
Still have to look at patch 2 and forth.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #34 from Marcel de Rooy  ---
Created attachment 48389
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48389=edit
Bug 13871 [QA Followup] - Rename parameter to send_msg_on_invalid_credentials

There are likely other services that would use this same feature.
Let's name it something a bit more general.

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #33 from Marcel de Rooy  ---
Created attachment 48388
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48388=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha…' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha…' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-



-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha…'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #32 from Marcel de Rooy  ---
Created attachment 48387
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48387=edit
Bug 13871: [TITLE_AMENDED] Additional changes

The original perltidy patch from Mason has been amended.

The perltidy itself has been moved to a separate patch with the current
perltidyrc applied.

As noted on Bugzilla, the original perltidy patch included some extra
changes:
[1] You prefix timestamp with Sip
This is not actually needed (it is imported), but if we should prefix it,
we should prefix now with C4::SIP::Sip. But you only changed two
occurrences (out of 26). So I remove these two changes.
[2] You remove the $server parameter from two calls of maybe_add:
A closer look at the remaining code tells me that $server is always
passed to maybe_add for FID_SCREEN_MSG. So this only left me the
current whitespace change.
But at least we documented what we did or did not, and why..

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #45646|0   |1
is obsolete||
  Attachment #45647|0   |1
is obsolete||
  Attachment #45648|0   |1
is obsolete||
  Attachment #45649|0   |1
is obsolete||

--- Comment #31 from Marcel de Rooy  ---
Created attachment 48386
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48386=edit
Bug 13871: Adjusted perl tidy on MsgType.pm

Run perltidy pro=xt/perltidyrc on the file.

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Marcel de Rooy  changed:

   What|Removed |Added

 CC||m.de.r...@rijksmuseum.nl

--- Comment #30 from Marcel de Rooy  ---
Some preliminary QA comment on the first patch only:

Smart perl tidy: Removing Sip:: and adding $server. If I only had such a smart
perl tidy :)
Seriously, we should not do ANY changes if we say that a patch is a perltidy
patch. Changes should be in another patch!
< $resp .= $lang . Sip::timestamp();
> $resp .= $lang . timestamp();
< $resp .= maybe_add( FID_SCREEN_MSG, $patron->screen_msg );
< $resp .= maybe_add( FID_SCREEN_MSG, $patron->{branchcode} )
> $resp .= maybe_add( FID_SCREEN_MSG, $patron->screen_msg, $server );
> $resp .= maybe_add( FID_SCREEN_MSG, $patron->{branchcode}, $server )
< $resp .= '' . ( ' ' x 10 ) . $lang . Sip::timestamp();
> $resp .= '' . (' ' x 10) . $lang . timestamp();

Interestingly, this cuddled else disappears too. But note that xt/perltidyrc
contains a --cuddled-else. Note that I do not like cuddled else, but somewhere
in time the community decided that we should use it. (Nothing better than
discussion about tabs or curly braces..)
-} else {
+}
+else {
Your tidy patch also contains constructs like (variable) instead of the
preferred ( variable ). Two simple examples:
< foreach my $i ( keys(%handlers) ) {
> foreach my $i (keys(%handlers)) {
< my ( $class, $msg, $seqno ) = @_;
> my ($class, $msg, $seqno) = @_;
So, it appears that you did not use the default Koha perltidyrc but some other
one.
By conclusion, I will add a new perl tidy patch and extract your real changes
in a separate patch.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

--- Comment #24 from Kyle M Hall  ---
> Nope, afaik, the '(1)' string is created and displayed completely within the
> Overdrive web app
> ..therefore, it is impossible to correct/remove this string within Koha

That seems like quite a coincidence, but I've no reason to doubt you! Back to
needs signoff!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #27 from Kyle M Hall  ---
Created attachment 45647
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45647=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha…' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha…' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-



-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha…'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #28 from Kyle M Hall  ---
Created attachment 45648
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45648=edit
Bug 13871 [QA Followup] - Rename parameter to send_msg_on_invalid_credentials

There are likely other services that would use this same feature.
Let's name it something a bit more general.

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #29 from Kyle M Hall  ---
Created attachment 45649
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45649=edit
Bug 13871 [QA Followup] - Fix namespaces

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #38452|0   |1
is obsolete||
  Attachment #38453|0   |1
is obsolete||
  Attachment #38454|0   |1
is obsolete||
  Attachment #38455|0   |1
is obsolete||

--- Comment #25 from Kyle M Hall  ---
Created attachment 45645
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45645=edit
Bug 13871 - perltidy build_patron_status

perltidy build_patron_status() sub

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall  changed:

   What|Removed |Added

  Attachment #45645|0   |1
is obsolete||

--- Comment #26 from Kyle M Hall  ---
Created attachment 45646
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45646=edit
Bug 13871 - perltidy build_patron_status

perltidy build_patron_status() sub

Signed-off-by: Kyle M Hall 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #23 from Mason James  ---
(In reply to Bob Birchall from comment #20)
> Given Mason's comments, does this still need to be in Failed QA?

any update Kyle?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #22 from Mason James  ---
(In reply to Kyle M Hall from comment #21)
> (In reply to Mason James from comment #19)
> > (In reply to Kyle M Hall from comment #11)
> > > I see. I hadn't looked deeply enough to realize 1 was the card number. In
> > > that case my solution won't work.
> > > 
> > > (In reply to Katrin Fischer from comment #10)
> > > > ... and 1 being the cardnumber.
> > 
> > just to confirm, (1) appears to be random string from within the Overdrive
> > app
> > ..it's not related to any 'id' within Koha
> 
> So for the patron status response, Koha is sending back the "(1)" but not
> for the patron information response?

Nope, afaik, the '(1)' string is created and displayed completely within the
Overdrive web app
..therefore, it is impossible to correct/remove this string within Koha

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #21 from Kyle M Hall  ---
(In reply to Mason James from comment #19)
> (In reply to Kyle M Hall from comment #11)
> > I see. I hadn't looked deeply enough to realize 1 was the card number. In
> > that case my solution won't work.
> > 
> > (In reply to Katrin Fischer from comment #10)
> > > ... and 1 being the cardnumber.
> 
> just to confirm, (1) appears to be random string from within the Overdrive
> app
> ..it's not related to any 'id' within Koha

So for the patron status response, Koha is sending back the "(1)" but not for
the patron information response?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #20 from Bob Birchall  ---
Given Mason's comments, does this still need to be in Failed QA?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #19 from Mason James  ---
(In reply to Kyle M Hall from comment #11)
> I see. I hadn't looked deeply enough to realize 1 was the card number. In
> that case my solution won't work.
> 
> (In reply to Katrin Fischer from comment #10)
> > ... and 1 being the cardnumber.

just to confirm, (1) appears to be random string from within the Overdrive app
..it's not related to any 'id' within Koha

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #18 from Mason James  ---
sorry, i missed some questions here...

(In reply to Kyle M Hall from comment #8)
> Pardon my being late to reply to this, but could this be solved with a a
> screen message regex?
> 
> 

^ no that wont work, the glitchy '(1)' string is created within the Overdrive
app, and external to Koha

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

 Status|In Discussion   |Failed QA

--- Comment #11 from Kyle M Hall k...@bywatersolutions.com ---
I see. I hadn't looked deeply enough to realize 1 was the card number. In that
case my solution won't work.

(In reply to Katrin Fischer from comment #10)
 ... and 1 being the cardnumber.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #16 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 38455
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38455action=edit
Bug 13871 [QA Followup] - Fix namespaces

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

  Attachment #38451|0   |1
is obsolete||

--- Comment #13 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 38452
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38452action=edit
Bug 13871 - perltidy build_patron_status

perltidy build_patron_status() sub

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

  Attachment #37539|0   |1
is obsolete||
  Attachment #37543|0   |1
is obsolete||

--- Comment #12 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 38451
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38451action=edit
Bug 13871 - perltidy build_patron_status

perltidy build_patron_status() sub

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #14 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 38453
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38453action=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha…' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha…' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-
accounts
login id=kohasip password=x delimiter=|
error-detect=enabled institution=YYY overdrive-mode=1 /
/accounts
-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha…'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #15 from Kyle M Hall k...@bywatersolutions.com ---
Created attachment 38454
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38454action=edit
Bug 13871 [QA Followup] - Rename parameter to send_msg_on_invalid_credentials

There are likely other services that would use this same feature.
Let's name it something a bit more general.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #17 from Kyle M Hall k...@bywatersolutions.com ---
This is looking really good, but I think if we are going to change the behavior
for the patron status response, I think we should have the same behavior for
the patron information response as well. Mason, can you implement that?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Kyle M Hall k...@bywatersolutions.com changed:

   What|Removed |Added

 CC||k...@bywatersolutions.com
 Status|Signed Off  |In Discussion

--- Comment #8 from Kyle M Hall k...@bywatersolutions.com ---
Pardon my being late to reply to this, but could this be solved with a a screen
message regex?

screen_msg_regex find=(1)  replace=Invalid patron or patron password /

If not, please reset the status of this bug.

(In reply to Bob Birchall from comment #0)
 When a Koha user offers correct credentials, Overdrive redirects to the
 users'
 Overdrive account page and no messages from Koha are displayed.
 
 However:
 1.  Where a correct cardnumber is entered with an incorrect password, Koha's
 SIP server responds with '(1) Greetings from Koha.'
 2.  Where an incorrect cardnumber is entered (with any password), the SIP
 server responds with a blank message: '(1) '
 
 These are displayed on an Overdrive page which includes a link: 'Click here
 to return to the sign in form' but there is nothing to draw the user's
 attention to their error and the need to follow that link back to try again.
 
 We propose to replace these messages with 'You have entered an incorrect
 cardnumber or password.  Please try again.'   This will appear just above
 the 'Click here to return ...' link and should make the process obvious for
 the user.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #10 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
... and 1 being the cardnumber.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #9 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hm, not sure this would work?

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha…' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha…' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #7 from Mason James m...@kohaaloha.com ---
(In reply to Katrin Fischer from comment #6)
 Hi Mason, 
 
 can the first patch be obsoleted?

hmm, no - it's a perltidy patch, that the feature depends on
as you can see from the tidy patch, the sub() desperately needed tidying

 I think this is not a problem specific to OverDrive, but to the error
 messages from the SIP server? Shoudl we adjust the title?

sure, thats fine with me

i really was thinking about calling the bug/patch something a bit more general.
but i couldn't really think of anything suitable


 To have the library show up in the release notes as a sponsor, you can add
 Sponsored-by: ... to the commit message
 (http://wiki.koha-community.org/wiki/Commit_messages)

thanks, i'll do that :0)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Katrin Fischer katrin.fisc...@bsz-bw.de changed:

   What|Removed |Added

Version|unspecified |master
 CC||colin.campbell@ptfs-europe.
   ||com
  Component|OPAC|SIP2

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Katrin Fischer katrin.fisc...@bsz-bw.de changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #6 from Katrin Fischer katrin.fisc...@bsz-bw.de ---
Hi Mason, 

can the first patch be obsoleted?
I think this is not a problem specific to OverDrive, but to the error messages
from the SIP server? Shoudl we adjust the title?
To have the library show up in the release notes as a sponsor, you can add
Sponsored-by: ... to the commit message
(http://wiki.koha-community.org/wiki/Commit_messages)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #3 from Mason James m...@kohaaloha.com ---
Created attachment 37539
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37539action=edit
Bug 13871 - OverDrive message when user authentication fails

perltidy build_patron_status() sub

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Mason James m...@kohaaloha.com changed:

   What|Removed |Added

  Attachment #37539|1   |0
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Mason James m...@kohaaloha.com changed:

   What|Removed |Added

  Attachment #37539|0   |1
is obsolete||

--- Comment #4 from Mason James m...@kohaaloha.com ---
Created attachment 37543
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37543action=edit
Bug 13871 - OverDrive message when user authentication fails

NOTE: apply this patch after the additional perltidy patch

this patch is basically a small work-around to fix some confusing login text,
when users enter incorrect auth details via Overdrive's website

with this option disabled (default) there is no change to SIP's behaviour

to test...

1/ configure your overdrive account to talk to your Koha's SIP service

2/ start Koha's SIP

3/ enter a correct username and correct password in overdrive
see overdrive display '(1) Greetings from Koha…' (good)

4/ enter a correct username and *incorrect* password in overdrive
see overdrive display '(1) Greetings from Koha…' (bad)

5/ enter an incorrect username in overdrive
see overdrive display '(1)' (badder)

6/ apply patch, enable 'overdrive-mode' in Koha's SIPConfig.xml

example...
-
accounts
login id=kohasip password=x delimiter=|
error-detect=enabled institution=YYY overdrive-mode=1 /
/accounts
-

7/ restart SIP

8/ enter a correct username and correct password
see overdrive display '(1) Greetings from Koha…'

9/ enter a correct username and *incorrect* password
see overdrive display '(1) Invalid patron or patron password'

10/ enter an incorrect username and incorrect password
see overdrive display '(1) Invalid patron or patron password'

http://bugs.koha-community.org/show_bug.cgi?id=1387

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Bob Birchall b...@calyx.net.au changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

--- Comment #5 from Bob Birchall b...@calyx.net.au ---
This is running in production for the sponsor, using 3.16.07.  It works nicely.
I'll sign off, but it may need another sign-off against Master?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Mason James m...@kohaaloha.com changed:

   What|Removed |Added

 Status|ASSIGNED|Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Bob Birchall b...@calyx.net.au changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Bob Birchall b...@calyx.net.au changed:

   What|Removed |Added

   Assignee|oleon...@myacpl.org |m...@kohaaloha.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

Bob Birchall b...@calyx.net.au changed:

   What|Removed |Added

  Change sponsored?|--- |Sponsored

--- Comment #1 from Bob Birchall b...@calyx.net.au ---
This change is being sponsored by Melbourne Athenaeum Library.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 13871] OverDrive message when user authentication fails

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13871

--- Comment #2 from Bob Birchall b...@calyx.net.au ---
Colin Campbell wrote:
I've had this reported recently but havent been able to look at it. I
think there might be a change of behaviour I think screen message used to
get a default value in cases like this rather than the cryptic 1. It
may be the initial flag value for screen_msg in ILS::Patron is not
getting replaced by a default?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/