Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ergänze classif_072 in step 2 #16

Closed
ToVie opened this issue Nov 5, 2013 · 4 comments
Closed

Ergänze classif_072 in step 2 #16

ToVie opened this issue Nov 5, 2013 · 4 comments
Assignees

Comments

@ToVie
Copy link

ToVie commented Nov 5, 2013

Bitte die Regel für classif_072 mit folgendem Element erweitern.

Vorschlag

<xsl:for-each select="$fragment/datafield[@tag='912']/subfield[@code='a']">
            <field name="classif_072">
                <xsl:value-of select="." />
            </field>
        </xsl:for-each>

P.S.: ich nehme an, dass es so geht, mach es aber nicht, da ich mir nicht sicher bin.

Grund

Da ich in swissbib nach MARC21 gültige 072-Codes von ungültigen trenne und letztere in 912 ablege, ist die Erweiterung nötig, um Fachcodes des IDS auch in in diesem Indexfeld ablegen zu können.

@ghost ghost assigned oschihin Nov 5, 2013
@oschihin
Copy link

@ToVie
Es gibt kein Feld 'classif_072'. In Template 'classifications' wird basierend auf $2 von 072 dynamisch eine Klassifikationsindexierung aufgebaut
https://github.com/swissbib/content2SearchDocs/blob/newCBS/xslt/swissbib.solr.vufind2.xsl#L309

Das führt offenbar nicht zu grossartigen Klassifikationen, siehe dazu den Schemabrowser von solr
Beispiel: http://search.swissbib.ch/solr/#/sb-biblio/schema-browser?field=classif_

Soweit ich sehe, hat 912 kein $2 mit der Quelle, Beispiel:

912         7   |a ac  

Oder ich kann sie ignorieren. In diesem Fall kann man das so einfach machen wie du vorschlägst, was ich jetzt gleich tun werde. Wir können uns das übermorgen zusammen anschauen. Ich nenne das Feld aber besser 'classif_912'.
Alternative: Indexierung sämtlicher 072 und 912 in ein Indexfeld, falls du das meintest

oschihin pushed a commit that referenced this issue Nov 12, 2013
* bezieht sich auf Issue #16
* indexiert unangemeldete Klassifikationen in eigenem Feld
@oschihin
Copy link

siehe auch #4

@ToVie
Copy link
Author

ToVie commented Nov 13, 2013

@oschihin aus meiner Sicht kann man sowohl ein eigenes Indexfeld für 912 machen als auch das ganze mit 072 zusammen ablegen.
Du hast recht, was $2 betrifft - das fehlt, da es nicht generisch befüllt wird, wenn es vom Verbund nicht geliefert wird. Es gibt also Fälle, bei denen es vorhanden ist (v.a. NEBIS), neben solchen, bei denen es fehlt (v.a. IDS Basel/Bern).

@oschihin
Copy link

In diesem Fall lasse ich das so wie oben gemacht, es gibt ein Indexfeld für 912 und mehrere dynamische für 072. Für 912 würde ich in dem Fall dynamische (basierend auf der Quelle) machen, wenn überall ein $2 gegeben wäre.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants