From bugzilla-daemon@bugs.koha-community.org Fri Oct 22 11:08:58 2021
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject: [Koha-bugs] [Bug 11175] Show the parent record's component parts in
 the detailed views
Date: Fri, 22 Oct 2021 09:08:56 +0000
Message-ID: <bug-11175-70-Gxz2NwJe9W@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-11175-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5312692496753482145=="

--===============5312692496753482145==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

--- Comment #425 from Jonathan Druart <jonathan.druart+koha@gmail.com> ---
Some patches have been squashed for readability.

(In reply to Marcel de Rooy from comment #390)
>  14 files changed, 322 insertions(+), 55 deletions(-)
>=20
> $showcomp variable naming; show_components better

Too late, too complicated or we add a separate commit for that. Not a big deal
IMO.

> Changes XSLTParse4Display; no tests. (Yes, there is no, but there should be=
.)

Still missing, I will try and provide some a bit later today.

> Code block if ($xslsyspref =3D~ m/Details/) {
> We are fetching components while we may not use them.
> Have the impression that the if inside the block should be incorporated into
> the main condition.

Done and squashed with the first patch.

> installer/data/mysql/atomicupdate/bug_11175.perl
> Needs to be converted to new style

I will do it when pushing it.

> +('ShowComponentRecords', 'nowhere', 'nowhere|staff|opac|both','In which
> record detail pages to show list of the component records, as linked via
> 773','Choice'),
> Should nowhere just be empty?

empty or 'nowhere' is fine.

(In reply to Marcel de Rooy from comment #391)
> What about indentation?

Done and squashed.

(In reply to Marcel de Rooy from comment #392)
> koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl
> koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>=20
> Seems you only remove 2 newlines, please remove these changes.

Done, squashed.

> +                            [% IF defaulttab =3D=3D 'components' %]
> +                            <li id=3D"tab_components" class=3D"ui-tabs-act=
ive">
> +                            [% ELSE %]
> +                            <li id=3D"tab_components">
> +                            [% END %]
> +                                <a href=3D"#components">Components ([% (
> ComponentParts.size ) %])</a>
> +                            </li>
>=20
> Indentation.

Done, squashed.

> diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slimUtils.xsl
> b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slimUtils.xsl
> index 105da508c7..2df0e1b3b4 100644
> --- a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slimUtils.xsl
> +++ b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slimUtils.xsl
> @@ -541,7 +541,6 @@
>              <xsl:value-of select=3D"$partName" />
>          </xsl:if>
>      </xsl:template>
> -
>  </xsl:stylesheet>
>=20
>  <!-- Stylus Studio meta-information - (c)1998-2002 eXcelon Corp.
>=20
> Dont do this please

Done, squashed.

(In reply to Marcel de Rooy from comment #393)
> my $components =3D $biblio->get_marc_components(300)
>=20
> This 300 is hardcoded. At least a constant? Or some other configuration
> option?
> Probably the latter.

Done, see follow-up.

(In reply to Marcel de Rooy from comment #394)
> Not from this report, but very related:
>=20
>         $self->{_components} =3D $results if ( defined($results) &&
> scalar(@$results) );
>     }
>=20
>     return $self->{_components} || [];
>=20
> Why do we save these component parts in the object and not use them later
> (if we would call again)?

Yes, that's a bad pattern. We discussed that recently but cannot remember
where.
We shouldn't cache that in the object IMO.

--=20
You are receiving this mail because:
You are watching all bug changes.

--===============5312692496753482145==--