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

Corrigido importação educacenso para tratar corretamente os ids inep e para escapar strings do endereço externo. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lucasdavila
Copy link

@cmsz

  • Em alguns casos os ids fornecidos pelo INEP são maiores que o maximo permitido para um valor inteiro (2147483647).
    • Por este motivo não pode-se usar funções como intval, ou o argumento %d junto a função sprintf.
    • Pois ids como 124669026568 seriam transformados em 2147483647.
  • A não correção deste erro, faria com que os registros cujo id inep é maior que 2147483647 sejam gravados com o mesmo id (2147483647).
  • Isto tambem causaria a duplicação de dados, uma vez que os scripts de importação não encontrariam um registro já importando com o id inep (uma vez que o id estaria gravado como 2147483647, não batendo com o id inep procurado.

  • Alem disso foi alterado a classe de endereço externo para escapar as strings, permitindo cadastrar endereços com palavras como "Floresta D'agua" por exemplo.

* Em alguns casos os ids fornecidos pelo INEP são maiores que o maximo
para um valor inteiro (2147483647).

  Por este motivo não pode-se usar funções como intval, ou o argumento
  %d junto a função sprintf.

  Pois ids como 124669026568 seriam transformados em 2147483647.

* A não correção deste erro, faria com que os registros cujo id inep é
  maior que 2147483647 sejam gravados com o mesmo id (2147483647).

* Isto tambem causaria a duplicação de dados, uma vez que os scripts de
  importação não encontrariam um registro já importando com o id inep
  (uma vez que o id estaria gravado como 2147483647, não batendo com o
  id inep procurado.
@lucasdavila lucasdavila changed the title Corrigido importação educacenso para tratar corretamente os ids inep. Corrigido importação educacenso para tratar corretamente os ids inep e para escapar strings do endereço externo. Mar 10, 2015
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

Successfully merging this pull request may close these issues.

None yet

1 participant