Skip to content

Conversation

@joffilyfe
Copy link

@joffilyfe joffilyfe commented Feb 11, 2020

O que esse PR faz?

Este pull request realiza alterações pontuais nos arquivos que de alguma maneira exibem os DOIs dos artigos. As páginas de texto completo, "como citar" e "nlinks" foram modificadas.

Onde a revisão poderia começar?

  • htdocs/xsl/sci_common.xsl L: 1098

Como este poderia ser testado manualmente?

Para testar este PR manualmente, deve-se:

  • Crie uma instância do SciELO metodologia [2];
  • Atualize as bases de acordo com seu teste (ex: as bases mst, xml de scielo br e medline);
  • Navegue entre artigos, verifique os links de DOI;
  • Verifique os link em "como citar";
  • Verifique os links na página arttext_plus;

Algum cenário de contexto que queira dar?

Não foram aplicados estilos em cascata para alterar a exibição, as cores, tamanhos ou fontes dos links. Sendo assim os links não terão uma aparência especial.

Screenshots

Screen Shot 2020-02-11 at 15 27 29
Screen Shot 2020-02-11 at 15 28 21

Quais são tickets relevantes?

N/A

Referências

[1] - Crossref DOI Guidelines_pt

Comandos

[2] - Criando uma nova instância de scielo metodologia

export SCIELO_FOLDER="$HOME/metodologia/dados-master"

docker run --name scielo-master -v "$SCIELO_FOLDER/scielo":/var/www/scielo \
    -v "$SCIELO_FOLDER/logs":/var/www/apache \
    --env INSTANCE_NAME=scielo_scl_master \
    --env USER_SUPERVISOR=root \
    --env PASS_SUPERVISOR=toor \
    --env SITE_NAME=localhost:8080 \
    --env USER_FTP=teste \
    --env PASSWD_FTP=teste \
    --env USER_PASS=scielo \
    --env GIT_BRANCH_NAME=master \
    -p 8080:8080 -p 2223:2222 -d scieloorg/metodologia:latest

@joffilyfe joffilyfe added the enhancement New feature or request label Feb 11, 2020
Este commit altera trechos de código que de alguma forma manipulam,
transformam e exibem os DOIs nas páginas de texto completo,
texto completo plus, "como citar" e "nlinks".

Seguindo as diretrizes do Crossref, os DOIs serão exibidos da
seguinte forma `https://doi.org/<doi>`. Também é necessário que o
texto seja representado como um link ( `<a>`).

Não foram aplicados estilos em cascata para alterar a exibição, as
cores, tamanhos ou fontes dos links.
@joffilyfe joffilyfe force-pushed the fix-doi-presentation branch from 7150840 to 1c07c6d Compare February 11, 2020 18:41
@gustavofonseca
Copy link
Contributor

Ainda não consigo aplicar o patch no meu repositório local.

gustavofonseca@srm-matriz-old:Web (rev708) git apply --check /tmp/708.patch
error: patch failed: cgi-bin/ScieloXML/sci_isoref.xis:274
error: cgi-bin/ScieloXML/sci_isoref.xis: patch does not apply
error: patch failed: cgi-bin/ScieloXML/sci_nlinks.xis:194
error: cgi-bin/ScieloXML/sci_nlinks.xis: patch does not apply
error: patch failed: cgi-bin/ScieloXML/sci_reflinks.xis:164
error: cgi-bin/ScieloXML/sci_reflinks.xis: patch does not apply
error: patch failed: htdocs/xsl/plus/data.xsl:347
error: htdocs/xsl/plus/data.xsl: patch does not apply
error: patch failed: htdocs/xsl/plus/layout.xsl:231
error: htdocs/xsl/plus/layout.xsl: patch does not apply
error: patch failed: htdocs/xsl/sci_common.xsl:1095
error: htdocs/xsl/sci_common.xsl: patch does not apply

@joffilyfe
Copy link
Author

@gustavofonseca não sei o porque deste erro no seu repositório local. Tentei reproduzir da seguinte forma:

  1. Clonei o repositório em um novo path;
  2. Verifiquei se o patch é aplicável joffily@lb-rancher ~/Web> git apply --check 708.patch (não deu erro);
  3. Apliquei git apply 708.patch;

O resultado foi:

joffily@lb-rancher ~/Web> git status                                                                                                                                                                                                                                   master!?
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   cgi-bin/ScieloXML/sci_isoref.xis
	modified:   cgi-bin/ScieloXML/sci_nlinks.xis
	modified:   cgi-bin/ScieloXML/sci_reflinks.xis
	modified:   htdocs/xsl/plus/data.xsl
	modified:   htdocs/xsl/plus/layout.xsl
	modified:   htdocs/xsl/sci_common.xsl

Untracked files:
  (use "git add <file>..." to include in what will be committed)

	708.patch

no changes added to commit (use "git add" and/or "git commit -a")

@gustavofonseca
Copy link
Contributor

@joffilyfe, acredito que havia alguma sujeira na minha cópia local. Após eu limpá-la a aplicação do patch passou a ser possível. Malz pelo alarme falso rs.

Copy link
Contributor

@gustavofonseca gustavofonseca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me parece bom =]

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants