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

[markup xml] Resolve problema de "mismatched" tags por conta das tags de estilo do Word #3234

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

robertatakenaka
Copy link
Member

O que esse PR faz?

Os XML gerados a partir do documento marcado pode estar quebrado por conta das tags de estilos que são geradas a partir dos estilos do Word e não a partir das etiquetas dos elementos do Markup.

Onde a revisão poderia começar?

Por commits

Como este poderia ser testado manualmente?

TODO...

Algum cenário de contexto que queira dar?

n/a

Screenshots

TODO

Quais são tickets relevantes?

#3201

Referências

n/a

Para primeiro passo, converte as tags "xml" em tags "sgml" de forma que sejam ignoradas ao carregar o xml em formato de árvore
Para segundo passo, verifica cada trecho node.text e tenta restaurar as tags de estilo, ou seja, convertendo de "sgml" para "xml" e verificando se o XML segue bem formado, caso contrário não restaura as tags dos trechos. As tags que causam um XML mal formado ficam como "sgml" para nos próximos passos serem tratadas.
Cria testes para a classe
…es nas extremidades até que não exista necessidade
…yle_tags_in_node_tail e seus respectivos conteúdo substituem as suas respectivas chamadas
…iar `_check_content`

e faz outros ajustes em `_update_node_text` e `_update_node_tail`
Elimina `_fix()` e seu conteúdo, ou seja, as chamadas a `_fix_inserting_tags_at_the_extremities` e `_fix_loading_xml_with_recover_true`, passam a ser feitas dentro de
`_restore_matched_style_tags_in_node_tails` e `_restore_matched_style_tags_in_node_texts`
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.

1 participant