Bug 8386 - Error in duplicate detection message for authorities
Summary: Error in duplicate detection message for authorities
Status: RESOLVED DUPLICATE of bug 8520
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: 3.8
Hardware: All All
: P5 - low normal
Assignee: Jared Camins-Esakov
QA Contact: Mason James
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-09 06:18 UTC by Katrin Fischer
Modified: 2012-09-10 19:02 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 8386: Error in duplicate detection message for authorities (1.22 KB, patch)
2012-07-09 12:01 UTC, Jared Camins-Esakov
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 8386: Error in duplicate detection message for authorities (1.31 KB, patch)
2012-07-09 19:33 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 8386: Error in duplicate detection message for authorities (1.26 KB, patch)
2012-09-09 18:41 UTC, Jared Camins-Esakov
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2012-07-09 06:18:43 UTC
If you create a new authority record and the name already exists in your database, you get a duplicate warning. This is good, but there is a small error in the message:

Duplicate record suspected

Is this a duplicate of HASH(0x47fa670)?
Comment 1 Jared Camins-Esakov 2012-07-09 12:01:24 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2012-07-09 19:33:59 UTC Comment hidden (obsolete)
Comment 3 Mason James 2012-07-17 03:55:54 UTC
passing QA

trivial patch, passes qa-tests OK

$ koha-qa.pl
starting perlcritic-progressive tests... OK
starting 'perl -c' syntax tests... OK
starting xt/tt_valid.t tests... OK
starting xt/author/vaild-template.t tests... OK
Comment 4 Mason James 2012-07-17 03:56:56 UTC
> This patch corrects the error message to show the title.
> 

a screen-shot would be nice here... :)
Comment 5 Chris Cormack 2012-07-18 08:00:06 UTC
Pushed to 3.8.x will be in 3.8.3
Comment 6 José Anjos 2012-08-01 21:50:39 UTC
I was trying to create a new Authority with the fowling parameters:
Lastname: Lopes,
Firstname: Francisco

I with version 3.08.03.000 I this error when I'm creating a new Authority and koha finds a possible duplicate:
Software error:
Can't use string ("Lopes, Francisco Luiz
<br/>
<br/") as a HASH ref while "strict refs" in use at /usr/share/koha/intranet/cgi-bin/authorities/authorities.pl line 613.


With version 3.08.01.000 its OK, there is no problem.

So I compare both functions and they are a bit different:
Ver: 3.08.01.000
<i>duplicateauthvalue  => $duplicateauthvalue,

Ver: 3.08.03.000
duplicateauthvalue  => $duplicateauthvalue->{authorized}->[0],

When I remove ->{authorized}->[0] it works OK.

-- 
José  Anjos
Comment 7 Jared Camins-Esakov 2012-09-09 18:41:43 UTC
Created attachment 12085 [details] [review]
Bug 8386: Error in duplicate detection message for authorities

If you create a new authority record and the name already exists in
your database, you get a duplicate warning. This is good, but there is
a small error in the message:

Duplicate record suspected
Is this a duplicate of HASH(0x47fa670)?

This patch corrects a reoccurence the error message to show the title.
Comment 8 Jared Camins-Esakov 2012-09-10 19:02:56 UTC
Tomas already fixed this, and the patch is signed off, I just missed it when I was looking yesterday.

*** This bug has been marked as a duplicate of bug 8520 ***