Errores varios y cuota por defecto al crear site(portal-ext.properties) #43
Conversation
| @@ -8,7 +8,7 @@ change-log= | |||
| page-url=http://www.liferay.com/community/user-groups/spain/welcome | |||
| author=LSUG | |||
| licenses=LGPL | |||
| liferay-versions=6.2.5+,6.2.10 | |||
| liferay-versions=6.2+ | |||
There was a problem hiding this comment.
Se puso explicitamente esta versión ya que creo que se usa API a partir de esta linea base. Esto no se cambiara.
There was a problem hiding this comment.
Ok, pues por lo menos, pon la versión 6.2.16(Que es la que estamos usando :( , te confirmo que funciona bien)
|
|
||
| QuotaLocalServiceUtil.createDefaultQuota(companyId, classNameId, group.getClassPK()); | ||
| QuotaLocalServiceUtil.decrementQuota(classNameId, group.getClassPK(), quotaUsed); | ||
|
|
There was a problem hiding this comment.
¿Por que es necesario decrementar la Quota aqui?
There was a problem hiding this comment.
Porque si tienes un site que ya usa recursos y instalas el portlet con una cuota por defecto, al inicializarse la cuota del site por defecto ya debe decrementar el uso que ya se esta haciendo de recursos. Sino, te encontrarías que al instalar el portlet cuota con una cuota por defecto, el uso de recursos hasta la fecha no se contabilizaría.
There was a problem hiding this comment.
Para eso ya esta la llamada a addQuota con la variable quotaUsed :-).
Has visto que esto no funcionaba bien?
There was a problem hiding this comment.
El método addQuota no tiene en cuenta la inicialización por defecto, si se cambia hay que cambiar el método. Por otro lado, entiendo que el método createDefaultQuota esta para eso, como mucho añadirle el parámetro para hacer en un solo paso. Como quieras...
There was a problem hiding this comment.
Si que tiene en cuenta la inicializacion por defecto. Como dije antes, para eso esta la variable quotaUsed en dicho método.
Es decir, los dos métodos que has puesto equivalen a la linea que ya existía.
Este cambio no es necesario ya que no cambia nada del comportamiento anterior.
There was a problem hiding this comment.
Juan no nos estamos entendiendo. El método addQuota no tiene en cuenta la configuración por defecto que AHORA puedes poner en el fichero portal-ext.properties. Esta modificación es muy necesaria. Mírate la definición de los métodos en la classe QuotaLocalServiceImpl:
public Quota createDefaultQuota(long companyId, long classNameId,
long classPK) throws SystemException {
Quota quota = quotaLocalService.fetchQuotaByClassNameIdClassPK(classNameId, classPK);
if (quota == null) {
int quotaAlert = 0;
long quotaAssigned = 0;
long quotaUsed = 0;
int quotaStatus = 0;
if ( PropsUtil.get("lsug.quota.portlet.default.alert")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.alert").equals("") ) quotaAlert=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.alert") );
if ( PropsUtil.get("lsug.quota.portlet.default.assigned")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.assigned").equals("") ) quotaAssigned=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.assigned") );
if ( PropsUtil.get("lsug.quota.portlet.default.status")!=null ) if ( !PropsUtil.get("lsug.quota.portlet.default.status").equals("") ) quotaStatus=Integer.parseInt( PropsUtil.get("lsug.quota.portlet.default.status") );
quota = quotaLocalService.addQuota(companyId, classNameId, classPK, quotaAlert,
quotaAssigned, quotaUsed, quotaStatus);
}
return quota;
}
Si quieres lo comentamos por teléfono.
There was a problem hiding this comment.
Ok, entendido! Pensare si ese es el mejor sitio.
Gracias.
acoronadoc
changed the title
| <struts-action> | ||
| <struts-action-path>/document_library/view_file_entry</struts-action-path> | ||
| <struts-action-impl>org.lsug.quota.QuotaEditFileEntryAction</struts-action-impl> | ||
| </struts-action> |
There was a problem hiding this comment.
Hola @acoronadoc. Estoy a punto de acabar el portlet para Liferay 7 y quería mirar lo que puedo añadir de esta pull request. ¿Por qué es necesario añadir el view_file_entry si no influye en la modificación de la cuota, y la nueva excepcion que lanza?
There was a problem hiding this comment.
Es necesario para que la cuota se actualice cuando utilizas la funcionalidad de arrastrar ficheros.
There was a problem hiding this comment.
¿Te refieres a arrastrar ficheros desde el ordenador a la pagina inicial de la document library?
There was a problem hiding this comment.
Hola de nuevo. He estado haciendo pruebas y en caso de cualquier error al arrastrar un fichero desde una carpeta del sistema operativo, no muestra ninguno (he abierto https://issues.liferay.com/browse/LPS-75051 por ello).
He probado a añadir este cambio y el comportamiento sigue igual (no muestra el mensaje personalizado de cuota por ningún sitio).
¿Puedes describir los pasos a reproducir? He probado tanto Liferay 7 GA4 como 6.2 GA5
There was a problem hiding this comment.
Hola Juan, diría que para arreglar esto solo hice este cambio(También hace varios meses de esto y arreglé otras cosas también). No me suena haber tocado nada mas.
There was a problem hiding this comment.
Vaya, pues en mi caso no soluciona nada. En tal caso no añadiré este cambio. Por ahora sólo añadiría la mejora de las propiedades por defecto y la traducción al catalán.
No description provided.