Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo #62

ManuSinergiaCRM · 2024-01-15T16:02:26Z

Closes Incidencia - Constructor - Ver nombre de fichero en vista de lista de un módulo propio #24
Issue en SA: In a file-based custom module, view file information in the subpanel and list view salesagility/SuiteCRM#10576

Este PR soluciona la incidencia descrita en el issue copiando el valor de la columna filename de la base de datos en el campo uploadfile del bean de un módulo.

Pruebas

Crear un módulo de tipo Archivo y una relación con Personas de tipo N a 1, es decir, que una persona pueda relacionarse con N registros de este módulo.
Publicar el módulo y cargarlo en el CRM.
Crear un registro e indicarle un archivo y una persona.
En la vista de lista del módulo, comprobar que, además del icono del ojo, se muestra el nombre del fichero y que al pulsarlo se muestra el cuadro de dialogo para su descarga
En la vista de subpanel, comprobar que se muestra el nombre del archivo, el icono del ojo y el icono de un fichero. Al pulsar el icono del ojo debería abrirse el archivo en una pestaña del navegador y al pulsar en el icono del fichero la opción de descarga.

github-actions · 2024-01-15T16:03:46Z

Actions executed at: 2024-09-24 08:19:30.

enricsinergia

Si es posible, el formato del subpanel debería ser más parecido al de documentos (queda más ordenado).

El icono de visualización me ha quedado un poco "colgado"

ManuSinergiaCRM · 2024-07-20T09:45:33Z

Lo estuve intentando pero hasta el momento solo lo conseguí con la vista de lista... Realizando pruebas conseguí que se mostrarán ambas opciones (descarga y visualización) en el subpanel y pienso que es un buen acercamiento.

Como indico en este (comentario), la estructura de los módulos es diferente y me da la sensación de que, además de modificar las vistas, se podría investigar por que la creación de un módulo de tipo Fichero no es más similar a la de Documentos

En consecuencia, mi primera opción sería quedarnos con esta mejora ya que cumple a nivel funcional y comunicárselo a SA para que valore revisar el proceso de creación de módulos de manera que al crear uno de tipo fichero se comporte como el de Documentos.

También puedo seguir investigando... A ver que dice Alberto 👍

ManuSinergiaCRM · 2024-08-01T10:36:20Z

Si es posible, el formato del subpanel debería ser más parecido al de documentos (queda más ordenado).

El icono de visualización me ha quedado un poco "colgado"

Buenas, he conseguido que el subpanel de un módulo nuevo de tipo Fichero se parezca bastante al de Documentos. Para ello he creado un widget y he modificado el layout que se genera en la creación de un módulo nuevo.

Hay una pequeña diferencia en el comportamiento de ambos subpaneles ya que el primer icono, en Documentos abre la vista de detalle del registro y en el módulo nuevo abre el enlace de descarga. Para igualar el comportamiento valdría con quitar las siguientes líneas del layout del subpanel pero de primeras no se mostrará icono ya que es necesario que el módulo nuevo tenga un icono propio. como por ahora los módulos propios no tienen icono, he dejado las siguientes líneas:

'image2_url_field' => array(
    'id_field' => 'id',
    'filename_field' => 'filename'
),

Esta solución solo aborda la incidencia de las vistas y pienso que podría ampliarse a corregir el proceso de creación de un módulo de tipo fichero. Propongo una de las siguientes opciones:

Crear un issue en el repo de SA e indicar que hay mucha diferencia en la estructura de campos y vistas de un módulo de tipo fichero y el de Documentos.
Crer un issue en nuestro repo y relacionarlo con este PR para deshacer los cambios que correspondan tras crear una estructura de campos nueva.

new changes

jordiSTIC · 2024-08-29T08:16:08Z

custom/include/generic/SugarWidgets/SugarWidgetSubPanelFileDownloadViewLink.php

+            || $layout_def['DetailView'] && !ACLController::moduleSupportsACL($layout_def['owner_module'])
+            || ACLController::checkAccess($layout_def['owner_module'], 'view', $layout_def['owner_id'] == $current_user->id, 'module',  $groupAccessView))) {
+            $link = ajaxLink("index.php?entryPoint=download&id={$record}&type={$module}");
+            return '<a href="' . $link . '" >'.$value.'</a><a href="' . $link . '&preview=yes" >&nbsp;&nbsp;&nbsp;&nbsp;<i class="glyphicon glyphicon-eye-open"></i></a>';


Propuesta: Añadir espacio entre el texto y el icono del ojo:

<i class="glyphicon glyphicon-eye-open" style="margin-left: 16px"></i>

jordiSTIC

Pruebas realizadas: el funcionamiento es correcto.
En el caso que el formato no sea "visualizable", se descarga.

Propuesta de cambios:

Añadir separación entre el nombre del fichero y el icono del ojo (parecen la misma entidad) Ver comentario con propuesta de cambio
Abrir la visualización del fichero en una nueva pestaña

ManuSinergiaCRM · 2024-09-12T12:30:56Z

Pruebas realizadas: el funcionamiento es correcto. En el caso que el formato no sea "visualizable", se descarga.

Propuesta de cambios:

Añadir separación entre el nombre del fichero y el icono del ojo (parecen la misma entidad) Ver comentario con propuesta de cambio

Abrir la visualización del fichero en una nueva pestaña

Implementado el punto 1

En relación al punto 2, me parece bien la propuesta pero haría un issue nuevo ya que ese campo está usando el widget SubPanelDetailViewLink y la modificación sería global.

Ese widget ya lo tenemos adaptado (https://github.com/SinergiaTIC/SinergiaCRM-SuiteCRM/pull/861) por lo que no estamos añadiendo complejidad a las actualizaciones. Sería modificar la siguiente línea para añadirle target=”_blank al elemento <a>

https://github.com/SinergiaTIC/SinergiaCRM/blob/develop/include/generic/SugarWidgets/SugarWidgetSubPanelDetailViewLink.php#L150

New changes

SinergiaCRM · 2024-09-18T12:35:08Z

Pruebas realizadas: el funcionamiento es correcto. En el caso que el formato no sea "visualizable", se descarga.
Propuesta de cambios:

Añadir separación entre el nombre del fichero y el icono del ojo (parecen la misma entidad) Ver comentario con propuesta de cambio

Abrir la visualización del fichero en una nueva pestaña

Implementado el punto 1

En relación al punto 2, me parece bien la propuesta pero haría un issue nuevo ya que ese campo está usando el widget SubPanelDetailViewLink y la modificación sería global.

Ese widget ya lo tenemos adaptado (SinergiaTIC/SinergiaCRM-SuiteCRM#861) por lo que no estamos añadiendo complejidad a las actualizaciones. Sería modificar la siguiente línea para añadirle target=”_blank al elemento <a>

https://github.com/SinergiaTIC/SinergiaCRM/blob/develop/include/generic/SugarWidgets/SugarWidgetSubPanelDetailViewLink.php#L150

En relación al punto 2:

Se detecta que la apertura de los enlaces en una pestaña nueva o no tiene relación con el entorno en el cuál estemos realizando las pruebas. En mi caso, en localhost los enlaces se abren (por defecto) en una pestaña nueva mientras que en Develop y ManuelSuite no.

Esto puede ser debido a la diferencia en el protocolo de conexión (HTTP y HTTPS) ya que algunos navegadores pueden restringir la forma en que se abren enlaces dependiendo de las configuraciones de seguridad de los entornos.
En consecuencia, interpreté que te referías al enlace del nombre pero al probar en Develop, ahora interpreto que te referías al ojo
Ya está actualizado que el ojo abra el contenido en una pestaña nueva.

Dejo anotado lo hablado con Alberto en relación al subpanel del módulo creado por la entidad:

El Icono de Documentos descargará el fichero
El Nombre de Documento abrirá la vista de detalle en la misma pestaña
El Nombre de Archivo descargará el fichero
El Icono del ojo abrirá el contenido del fichero en una pestaña nueva

enricsinergia

(A)provado

jordiSTIC

(A)Probado

Lógica para mapear filename y uploadfile

ced96cb

ManuSinergiaCRM self-assigned this Jan 15, 2024

ManuSinergiaCRM added bug Something isn't working Administración del CRM not urgent labels Jan 15, 2024

Asegurarnos de que procede aplicar la lógica

927a71e

ManuSinergiaCRM changed the title ~~Lógica para mapear filename y uploadfile~~ Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo Jan 15, 2024

ManuSinergiaCRM added 6 commits January 15, 2024 18:19

Añadir PR Id

2a7dcb5

wip

a484c90

wip

b33dae5

wip

714e4db

wip

66e61ef

wip

4e453cc

AlbertoSTIC force-pushed the develop branch from 98edf5b to 8d6d4a1 Compare January 18, 2024 12:25

ManuSinergiaCRM added 2 commits July 15, 2024 09:03

Merge branch 'develop' into bug/viewFileNameInListView

bb15f18

Modificar las plantillas de Vista de lista y vista de subpanel

bd1bf2a

ManuSinergiaCRM closed this Jul 15, 2024

ManuSinergiaCRM reopened this Jul 15, 2024

ManuSinergiaCRM requested a review from AlbertoSTIC July 15, 2024 12:19

AlbertoSTIC requested review from enricsinergia and jordiSTIC and removed request for AlbertoSTIC July 17, 2024 12:52

enricsinergia previously requested changes Jul 19, 2024

View reviewed changes

ManuSinergiaCRM added 2 commits July 29, 2024 13:28

Merge branch 'develop' into bug/viewFileNameInListView

b9bd0fb

Estandarizar layout de subpanel para que se parezca al de documentos

d5b20a7

ManuSinergiaCRM requested review from enricsinergia and removed request for jordiSTIC August 1, 2024 10:36

ManuSinergiaCRM requested a review from jordiSTIC August 1, 2024 10:36

jordiSTIC reviewed Aug 29, 2024

View reviewed changes

jordiSTIC previously requested changes Aug 29, 2024

View reviewed changes

Añadir separación entre el nombre del fichero y el icono del ojo

3fdab0e

ManuSinergiaCRM requested a review from jordiSTIC September 12, 2024 12:31

Abrir en pestaña nueva al pulsar en el enlace del ojo

6b268bf

enricsinergia approved these changes Sep 23, 2024

View reviewed changes

jordiSTIC approved these changes Sep 23, 2024

View reviewed changes

jordiSTIC merged commit 6a0e587 into develop Sep 24, 2024
1 check passed

jordiSTIC deleted the bug/viewFileNameInListView branch September 24, 2024 08:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo #62

Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo #62

ManuSinergiaCRM commented Jan 15, 2024 •

edited

Loading

github-actions bot commented Jan 15, 2024 •

edited

Loading

enricsinergia left a comment

ManuSinergiaCRM commented Jul 20, 2024

ManuSinergiaCRM commented Aug 1, 2024

jordiSTIC Aug 29, 2024

ManuSinergiaCRM Sep 12, 2024

jordiSTIC left a comment

ManuSinergiaCRM commented Sep 12, 2024

SinergiaCRM commented Sep 18, 2024 •

edited

Loading

enricsinergia left a comment

jordiSTIC left a comment

Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo #62

Hotfix - General - Ver nombre de fichero en vista de lista de un módulo propio de tipo Archivo #62

Conversation

ManuSinergiaCRM commented Jan 15, 2024 • edited Loading

github-actions bot commented Jan 15, 2024 • edited Loading

enricsinergia left a comment

Choose a reason for hiding this comment

ManuSinergiaCRM commented Jul 20, 2024

ManuSinergiaCRM commented Aug 1, 2024

jordiSTIC Aug 29, 2024

Choose a reason for hiding this comment

ManuSinergiaCRM Sep 12, 2024

Choose a reason for hiding this comment

jordiSTIC left a comment

Choose a reason for hiding this comment

ManuSinergiaCRM commented Sep 12, 2024

SinergiaCRM commented Sep 18, 2024 • edited Loading

enricsinergia left a comment

Choose a reason for hiding this comment

jordiSTIC left a comment

Choose a reason for hiding this comment

ManuSinergiaCRM commented Jan 15, 2024 •

edited

Loading

github-actions bot commented Jan 15, 2024 •

edited

Loading

SinergiaCRM commented Sep 18, 2024 •

edited

Loading