https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15048 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl Status|Signed Off |Failed QA QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | Patch complexity|--- |Small patch --- Comment #16 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment: Thanks for your patch, Mason. I think it still needs some attention: [1] You add $x when indexing 655. Please note that the whole field 655 is already indexed under term Subject. (So you would only add to Index-term-genre.) But the XSLT just looks at Subject (and only with some specific settings for UseAuthoritiesForTracings and TraceSubjectSubdivisions). Normally you would search for the authority number (an). So, it does not seem to be necessary to add it. Please explain why you need it. [2] If it should be necessary, what about $v $y and $z ? -- You are receiving this mail because: You are watching all bug changes.