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

Bug na validação de page-count do Style Checker #31

Closed
lpanebr opened this issue Mar 10, 2015 · 3 comments
Closed

Bug na validação de page-count do Style Checker #31

lpanebr opened this issue Mar 10, 2015 · 3 comments
Assignees

Comments

@lpanebr
Copy link

lpanebr commented Mar 10, 2015

Oi Gente,

Alguns periódicos usam as páginas do Editorial prefixadas (exemplo: E1) e isso gera erro na validação do /article/front/article-meta/counts/page-count pois fpage e lpage não são inteiros.[1]

O erro está na linha 455.

A solução (para PREFIXOS) seria considerar apenas os números à direita dos elementos <fpage>E1</fpage> e <lpage>E1</lpage>.

hmmmm... temos também os SUFIXOS... embora mais raros. Talvez a solução mais genérica seja capturar apenas o número mesmo, descartando eventuais prefixos e sufixos:

 [^\d]*(\d+)[^\d]*

O trecho relevante do XML:

  <front>
    <article-meta>
[...]
      <pub-date pub-type="epub-ppub">
        <season>Jan-Feb</season>
        <year>2015</year>
      </pub-date>
      <volume>25</volume>
      <issue>1</issue>
      <fpage>E1</fpage>
      <lpage>E1</lpage>
      <counts>
        <fig-count count="0"/>
        <table-count count="0"/>
        <equation-count count="0"/>
        <ref-count count="0"/>
        <page-count count="1"/>
      </counts>
    </article-meta>
  </front>

[1] Exemplo de saída com erro Element 'counts': Missing element or wrong value in page-count.
image

@gustavofonseca
Copy link
Contributor

Luciano, grato por reportar o problema. Você identificou apenas em editoriais?

On 10/03/2015, at 09:58, Luciano notifications@github.com wrote:

Oi Gente,

Alguns periódicos usam as páginas do Editorial prefixadas (exemplo: E1) e isso gera erro na validação do /article/front/article-meta/counts/page-count pois fpage e lpage não são inteiros.[1]

O erro está na linha 455 https://github.com/scieloorg/packtools/blob/master/packtools/catalogs/scielo-style-1.1.sch#L455.

A solução (para PREFIXOS) seria considerar apenas os números à direita dos elementos E1 e E1.

hmmmm... temos também os SUFIXOS... embora mais raros. Talvez a solução mais genérica seja capturar apenas o número mesmo, descartando eventuais prefixos e sufixos:

[^\d](\d+)[^\d]
O trecho relevante do XML:

[...] Jan-Feb 2015 25 1 E1 E1 [1] Exemplo de saída com erro Element 'counts': Missing element or wrong value in page-count. https://cloud.githubusercontent.com/assets/129185/6574982/1cce2642-c709-11e4-81e9-aa6ac5804c89.png — Reply to this email directly or view it on GitHub https://github.com//issues/31.

@lpanebr
Copy link
Author

lpanebr commented Mar 10, 2015

De nada Gustavo.
O problema ocorre em artigos também.

@gustavofonseca gustavofonseca self-assigned this Mar 10, 2015
@gustavofonseca
Copy link
Contributor

Na realidade só é possível checar automaticamente o valor de page-count quando fpage e lpage são dígitos.

gustavofonseca pushed a commit that referenced this issue Mar 16, 2015
A premissa de que os valores válidos para *//article-meta/fpage* e
*//article-meta/lpage* são dígitos é inválida. Portanto, nesse caso
o valor de *//counts/page-count* não pode ser validado automaticamente.
jfunez pushed a commit to jfunez/packtools that referenced this issue Jun 23, 2017
…17-05-10_sub-article_history

Apresenta o link para history de sub-article quando não é translation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants