Menu

#169 editor libreria documentos Liferay

v4m6
closed
nobody
None
6
2010-09-17
2010-09-08
Oscar Caro
No

Este editor permite asociar una colección de documentos a una entidad. Los documentos se almacenarán en la librería de documentos de Liferay bajo la carpeta raíz que se configure en el editor.
Lógicamente, el código tiene una parte dependiente de la API de Liferay y ha sido probrado sólo en Liferay 5.2.3. En versiones anteriores que no usaban el patrón "ServiceContext" de Liferay no funcionaría

Discussion

  • Oscar Caro

    Oscar Caro - 2010-09-08
     
  • Javier Paniza

    Javier Paniza - 2010-09-09
    • priority: 5 --> 6
     
  • Javier Paniza

    Javier Paniza - 2010-09-13

    He estado revisando el código y la verdad es que está bastante bien. Listo para descomprimir en cualquier proyecto OpenXava y empezar a usarlo. Como "plugin" es suficiente, pero si queremos integrarlo en OpenXava aún debemos hacer algunos refinamientos, más que nada por coherencia de estilo con el resto de OpenXava.
    Estos son las correcciones que te propongo para poder integrarlo con OX:
    - Los paquetes deberían ser org.openxava y no org.gava.
    - Los comentarios del código deben de estar en inglés.
    - Sobre FolderCalculator: debería ser una interfaz y no una clase abstracta, algo así como IFolderCalculator. Después puedes tener una clase FolderCalculatorBase si quieres. Más aun, en lugar de crear IFolderCalculator deberías usar ICalculator, porque el sufijo Calculator tiene cierta connotaciones en OX (como tener método calculate()). No puedes llamar calculator a algo que no sea un calculator. Además, el @Inject solo funciona en las acciones.
    - Hay referencias a clases que no están ni en OpenXava/lib ni en OpenXava/WEB-INF/lib. Has de proveer lo jars necesarios para poder compilar las clases.

     
  • Oscar Caro

    Oscar Caro - 2010-09-14

    con los cambios sugeridos

     
  • Oscar Caro

    Oscar Caro - 2010-09-14

    He subido un zip nuevo con los siguientes cambios:
    - Los paquetes son ahora org.openxava
    - Los comentarios están en inglés
    - En cuanto a FolderCalculator: Calcular dinámicamente la carpeta raíz era una funcionalidad que al inicio del proyecto pensábamos que necesitaríamos, pero finalment no fue así y no está probada. Si además no podemos inyectar el objeto View, creo que es poco útil. Por este motivo he quitado este parámetro en la nueva versión. Si consideras que sería interesante tener esta característica, lo reprogramaría teniendo en cuenta las sugerencias de estilo openxava que me indicas.
    Con respecto a las referencias, se necesitan dos jars portal-kernel.jar y portal-service.jar de Liferay (5.2.3), pero no me deja subirlos porque son demasiado grandes, ¿cómo te los paso?
    Por último en dlUploadFile.jsp habría que internacionalizar "Select files". Supongo que se hace con XavaResources y añadiendo el recurso en los diferente idiomas, ¿debo hacerlo yo?

     
  • Javier Paniza

    Javier Paniza - 2010-09-17

    Oscar,
    ya he integrado tu código en OpenXava estará disponible en 4m6.

    He renombrado las clases y JSP (pero no el estereotipo ni el editor), para que aparezca la palabra Liferay, más que nado para dejar patente que el editor sólo funciona con Liferay.

    El único cambio de código que he hecho es un pequeño refinamiento en la acción que carga el fichero: cuando no había usuario identificado o no estaba creada la carpeta raiz para los documentos se lanzaba una NullPointerException, en la que la única pista era el número de línea de la acción. Lo he cambiado para que en estos casos aparezca un mensaje significativo.

    Todo lo demás lo he dejado tal cual.

    Como te he dicho ya está integrado en OX, pero además adjunto el código modificado a este hilo por si te interesa verlo.

    ¿Te parece bien que ponga tu nombre los créditos de OpenXava?
    ¿Quieres que ponga un vínculo al ayuntamiento en la página de créditos?

     
  • Javier Paniza

    Javier Paniza - 2010-09-17
     
  • Javier Paniza

    Javier Paniza - 2010-09-17
    • milestone: --> v4m6
    • status: open --> closed
     
  • Oscar Caro

    Oscar Caro - 2010-09-21

    Javier,
    Mejor que mi nombre, puedes poner Ayuntamento de Gavá y enlazar a www.gavaciutat.cat

    Esperamos poder hacer más contribuciones en un futuro próximo.

     
  • Javier Paniza

    Javier Paniza - 2010-09-21

    Bueno, pues ya sois parte del "equipo de OpenXava":
    http://www.openxava.org/web/guest/credits

     

Log in to post a comment.