Correc. - Mejora #3459
cerradaProyecto / documento por etiquetas: al modificar un documento cambian los iconos de accion por documento
0%
Descripción
Los documentos en el listado de documentos por etiquetas no se pueden borrar, y solo se pueden modificar aquellos que no estan vinculados a gastos, transferencias, o fuentes de verificación.
Al editar uno y guardar en el refresco de la linea del documento cambian los iconos de las acciones para cada documento y permite borrar el documento.
Actualizado por Jaime Ortiz hace alrededor de 6 años
Existe un fallo de diseño. Dado que la edicion de los documentos del listado de documentos por etiquetas se hace reutilizando los metodos del controlador documento_controller.rb pero sin añadirle ningun comportamiento especifico para esta situacion.
Asi en la vista app/views/documento_busqueda/listado.html.erb se llama:
11 <% url_editar = {:controller => :documento, :action => 'editar_nuevo', :id => documento, :update => documento.id, :update_listado => (update_listado ||= nil)} % >
De hecho no solo existe ese problema si no que en otras pruebas estan saltando otro errores, en la llamada:
Processing by DocumentoController#editar_nuevo as JS Parameters: {"authenticity_token"=>"SR4E0ejQhHYGFRLBxHYavze2RCQjkVC1+PGFd287QUb/P/l7v4A76QHsH454pmgwk5/ND6wxjIceILNdglbiZg==", "update"=>"5", "seccion"=>"proyectos", "proyecto_id"=>"2", "menu"=>"documentos_proyecto", "id"=>"5"}
salta:
NoMethodError (undefined method `escritura_permitida' for nil:NilClass): app/models/documento.rb:104:in `escritura_permitida' app/controllers/documento_controller.rb:52:in `autorizar_documento'
Al no haber ningun espacio en al session. Habria que darle el comportamiento adecuado para este nuevo listado tanto si se reutiliza el metodo del controlador documento_controller.rb, como si no.
De momento como solución provisional deshacemos la edición de documentos desde el listado de documentos por etiquetas.
https://gong.org.es/projects/gor/repository/revisions/172ab40ce878bc365e1a5495234a6019b91ede8d
Actualizado por Jaime Ortiz hace alrededor de 6 años
- Estado cambiado de Nueva a Resuelta