Corrige vários bugs no prefixo oai-dc-scielo#752
Conversation
…ence> Há situações em que este campo está povoado com marcações <a> e isso acarreta falha na renderização do XML
|
|
||
| <xsl:if test="$value != ''"> | ||
| <xsl:value-of select=" concat( '<', $name, ' ', $param_name, '= "', $param_value, '" >', $value, '</', $name, '>' )" disable-output-escaping="yes"/> | ||
| <xsl:value-of select=" concat( '<', $name, ' ', $param_name, '="', $param_value, '">', $value, '</', $name, '>' )" disable-output-escaping="yes"/> |
There was a problem hiding this comment.
Será que o que vc quer seria:
<xsl:element name="{$name}">
<xsl:attribute name="{$param_name}"><xsl:value-of select="$param_value"/></xsl:attribute>
</xsl:element>
There was a problem hiding this comment.
@robertatakenaka, na sua sugestão faltaria o campo $value. Ficaria assim:
<xsl:element name="{$name}">
<xsl:attribute name="{$param_name}">
<xsl:value-of select="$param_value"/>
</xsl:attribute>
<xsl:value-of select="$value"/>
</xsl:element>
Que serviria pra gerar, por exemplo, as seguintes estruturas:
<issn publication-format="print">0066-782X</issn>
<issn publication-format="online">1678-4170</issn>
A alteração que fiz foi apenas para remover um espaço em branco que havia antes dos termos "print" e "online" (e em qualquer outro campo parametrizado). Estavam como "= print" e "= online".
robertatakenaka
left a comment
robertatakenaka
left a comment
There was a problem hiding this comment.
Verifique meus comentários
@robertatakenaka Substituí aquele código que gera a estrutura parametrizada. |
| <xsl:attribute name="{$param_name}"> | ||
| <xsl:value-of select="$param_value"/> | ||
| </xsl:attribute> | ||
| <xsl:value-of select="$value" disable-output-escaping="yes" /> |
There was a problem hiding this comment.
Talvez não seja desejável ser yes, pois > poderia ser convertido a >, certo? Pode confirmar?
There was a problem hiding this comment.
Eu preciso que converta. Veja um exemplo com o parâmetro disable-output-escaping
valorado com no
valorado com yes
Nós queremos que seja convertido, portanto, deve estar valorado com yes.
2 participants
O que esse PR faz?
Corrige seis bugs identificados na aplicação OAI-PMH quando utilizada com o prefixo
oai_dc_scielo. Além disso, adiciona uma base gizmo para tratar caracteres inválidos. A lista de bugs lista de bugs é composta por:>não tratado que gera XML com defeitooai:scielo:antes do identificador existente no campo<header>named-contentgrafado incorretamente dentro do campo<contrib-group>e em campo relacionado a Lattes.numde campo<reference>possui elementos inválidosOnde a revisão poderia começar?
Por commits.
Como este poderia ser testado manualmente?
3e observar as seguintes diferenças em relação ao obtido em2:>oai:scielo:no campoidentifierdoheaderdo documento.<dc:identifier pub-id-type="publisher-id">S0037-86822012000300020</dc:identifier><dc:identifier pub-id-type="publisher-id">oai:scielo:S0037-86822012000300020</dc:identifier>named-contentnos campos relacionados a identificadores Lattes e acontrib-groupname-contentnamed-content=de atributos de marcações.<issn publication-format= "print" ><issn publication-format="print">Algum cenário de contexto que queira dar?
N/A
Screenshots
Bug 1 (caractere sem escape aplicado)
Bug 3 (grafia incorreta)
Bug 4 (espaços inválidos)
Bug 7 (caractere inválido)
Bug 8 (documento Press Release sem informação de Issue)
Quais são tickets relevantes?
N/A
Referências
N/A