This manifests in the staff client
Created attachment 3138 [details] [review] Proposed Patch
Created attachment 3139 [details] [review] Followup patch
This bug is mentioned in: Bug 5749 Fix borrower address display in intranet http://lists.koha-community.org/pipermail/koha-patches/2011-February/013842.html Bug 5749 follow up patch, whitespace cleanup http://lists.koha-community.org/pipermail/koha-patches/2011-February/013843.html Bug 5749 Fix borrower address display in intranet http://lists.koha-community.org/pipermail/koha-patches/2011-February/013852.html
This has been signed off by Salvador Zaragoza Rubio
Pushed for QA new/bug_5749
I'm not clear what the nature of the problem is, so it's difficult for me to evaluate the patch. Could a summary of the display breakage be given, as well as the circumstances required to bring it about? Screenshots encouraged.
Paul, (or someone from biblibre) any more information on this?
BibLibre Mantis number = 1416 (reported by SAN-OP) lorsqu'on utilise les zones de saisie numéro de voie et type de voie, l'adresse n'est pas correctement affichée sur la liste de recherche d'un adhérent ainsi que sur la fiche de l'adhérent => when you use streetnumber / streetype, the address is not correctly displayed on the patron result list and on the borrower detail. HTH
Created attachment 4843 [details] [review] proposed patch (updated) QA comment * patch 1 contained a reference to .tmpl (removing an empty line). I've discarded this part of the patch * if your patron address, uses streetnumber and street type, were not displayed in result list or in detail before this patch, and are correctly displayed after. So this patch is OK * Major problem: It seems that the SetMemberInfosInTemplate sub added in C4/Members is NOT USED. So it should be removed. * Minor problem: The address on the left tab still does not contain streetnumber/type, it could be worth adding it here too Conclusion : * Marking Failed QA * will attach the patch with .tmpl removed * will affect the bug to Stephane Delaune, that submitted it (but is in holiday for 3 weeks) => Stéphane, please check if SetMemberInfosInTemplate is relevant or no, and argue or propose another patch.
Comment 8 still does not provide a clear explanation of what the problem actually is. What does "not correctly displayed" mean? How does it display now, and how SHOULD it display? What data in what order? Screenshots would be helpful.
Ian: if you use street type and streetnumber, an address like "108 rue Breteuil" has 108 in streetnumber rue in streettype Breteuil in address Before this patch the display is BibLibre Breteuil 13006 Marseille After this patch the display is correct : BibLibre 108 rue Breteuil 13006 Marseille If you store everything in address, the patch has no consequence, it's displayed "108 rue Breteuil" as address="108 rue Breteuil" and streetnumber/streettype are empty Is it clear now ?
Paul, Yes, very clear now, thank you. Patch can now be evaluated against a test condition.
Is attachment 3139 [details] [review] obsoleted by attachment 4843 [details] [review]? Many of the changes in attachment 4843 [details] [review] are also in the patch for Bug 5436. I'll consider this the feature-specific bug for the address display changes and remove them from Bug 5436.
Created attachment 8034 [details] [review] proposed patch (updated) In effect, the patch of Bug 5436 (which is pushed in master) seems to make this bug almost obsolete. As Paul wrote, if you use street type and streetnumber, an address like "108 rue Breteuil" has : "108" in streetnumber, "rue" in streettype, "Breteuil" in address With the Bug 5436 the display is now "108 rue, Breteuil" instead of "108 rue Breteuil" this little patch corrects it (remove the comma).
Created attachment 8126 [details] [review] [SIGNED-OFF] Bug 5749 Fix borrower address display in intranet Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I notice there are other problems with displaying addresses this way. For instance, circulation.pl and moremember.pl both use the same include file (circ-menu.inc), but only in circulation.pl does the [% address %] variable get built using street type, street number, and address: my $address = $borrower->{'streetnumber'}.' '.$roadttype_hashref->{$borrower->{'streettype'}}.' '.$borrower->{'address'}; I wonder why we're not doing this in the template instead?
QA comment: tiny patch, passed QA (In reply to comment #16) > I notice there are other problems with displaying addresses this way. For > instance, circulation.pl and moremember.pl both use the same include file > (circ-menu.inc), but only in circulation.pl does the [% address %] variable > get built using street type, street number, and address: > > my $address = $borrower->{'streetnumber'}.' > '.$roadttype_hashref->{$borrower->{'streettype'}}.' '.$borrower->{'address'}; > > I wonder why we're not doing this in the template instead? I think there's no specific reason to do it that way. Maybe that's a topic for next week, and defining "object" that will make a consistent display
follow-up pushed directly onto master, as the original branch new/bug_5749 was pre-3.4, so HTML::Template and the patch can't apply anymore
This bug will be included in the Koha 3.6.5 release.