Comment 3 for bug 1235497

Thanks, Kathy, for the feedback. I am glad we are on the same page.

Okay, and... ugh! It looks like we are going to need another custom MODS title field to pull this off. In testing, we really need both subtitle punctuation and nonSort separation in a single field in order to make browse work optimally. titleNonfiling gives us the subtitle punctuation, and titleInfo gives us the nonSort tag, but nothing gives us both. So, I propose adding something like:

            <titleBrowse>
                <xsl:variable name="titleBrowseChop">
                    <xsl:call-template name="chopPunctuation">
                        <xsl:with-param name="chopString">
                            <xsl:call-template name="subfieldSelect">
                                <xsl:with-param name="codes">abfgk</xsl:with-param>
                            </xsl:call-template>
                        </xsl:with-param>
                    </xsl:call-template>
                </xsl:variable>
                <xsl:choose>
                    <xsl:when test="@ind2>0">
                        <nonSort>
                            <xsl:value-of select="substring($titleBrowseChop,1,@ind2)"/>
                        </nonSort>
                        <title>
                            <xsl:value-of select="substring($titleBrowseChop,@ind2+1)"/>
                        </title>
                    </xsl:when>
                    <xsl:otherwise>
                        <title>
                            <xsl:value-of select="$titleBrowseChop"/>
                        </title>
                    </xsl:otherwise>
                </xsl:choose>
                <xsl:call-template name="part"></xsl:call-template>
            </titleBrowse>

I'm going to go ahead and make a branch for all this, but if this seems like the wrong path, it isn't too late to speak up.