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

Hotfix - Administración - Error en la edición, clonación y cancelación de un campo HTML en Estudio #464

Merged
merged 4 commits into from
Nov 19, 2024

Conversation

ManuSinergiaCRM
Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM commented Oct 29, 2024


Descripción

Este PR deshace los cambios implementados en el PR #388 ya que el código implementado en dicho PR era el que generaba el error detectado en el issue relacionado (#400). Además, implementa una solución alternativa para dar solución a la incidencia relacionada con el PR 388 (#81)

En este PR se ha intentado progresar con el enfoque anterior (mceRemoveControl) pero no se consigue dar solución a las diferentes casuísticas con el editor de código. Así, y viendo que con el botón Cancelar se quiere conseguir el mismo comportamiento que el aspa de cierre (X) de la pestaña donde se muestra el editor, se modifica el comportamiento del botón Cancelar para que al ser pulsado, llame a la función click() del aspa de cierre de pestaña.

Para resolver la incidencia con la clonación de campos HTML, se implementa la llamada al método mceRemoveControl de tinymce al pulsar en el botón, de forma que se elimine la relación entre el editor y el input htmlarea

`if (tinyMCE.get("htmlarea")) { tinymce.execCommand("mceRemoveControl", false, "htmlarea"); }`

Pruebas

Rehacer pruebas del PR #388

  1. Acceder a Estudio, crear un campo, seleccionar que sea de tipo HTML, asignarle un valor y comprobar que se muestra el editor enriquecido.
  2. Pulsar en el botón de Clonar y comprobar que se muestra el editor enriquecido.
  3. Volver a editar el campo creado en el punto 1 + pulsar en el botón de Cancelar
  4. Editar el campo y comprobar que se muestra el editor enriquecido.
  5. Asignar un valor diferente al que asignamos en el primer punto + Guardar
  6. Editar de nuevo el campo y comprobar que se ha guardado el valor indicado en el punto anterior

Pruebas

  1. Acceder a Estudio y crear dos campos de tipo HTML asignándoles un valor por defecto diferente
  2. Acceder al primer campo y pulsar en el botón de Cancelar
  3. Acceder al segundo campo y comprobar que se muestra su valor y no el valor del primer campo.

Copy link

github-actions bot commented Oct 29, 2024

Actions executed at: 2024-11-19 15:45:15.

Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

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

(A)Probado

Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

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

(A)probado

Se observa que si se modifica el valor por defecto antes de dar al botón clonar, se pierde la modificación, pero es lo mismo que sucede con el resto de tipos de campos (se copia el valor guardado en BBDD y no el que se haya modificado por pantalla)

@AlbertoSTIC AlbertoSTIC merged commit 88b996d into develop Nov 19, 2024
1 check failed
@AlbertoSTIC AlbertoSTIC deleted the bug/showAndCloseTinyMCEInStudio branch November 19, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Administración del CRM bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - Administración - Error al recuperar el valor por defecto de un campo HTML en Estudio
4 participants