fix: handle null FileEntry on riparaPermessiPareriGeologi method

liferay-plugins-sdk-6.2/hooks/sicilia-p7m-preview-hook/docroot/WEB-INF/src/com/liferay/portal/configuration/easyconf/ClassLoaderComponentConfiguration.java

@@ -76,6 +76,10 @@ public class ClassLoaderComponentConfiguration extends ComponentConfiguration {
 	public ComponentProperties getProperties() {
 		ComponentProperties componentProperties = _getAvailableProperties();
 
+		if(componentProperties == null) {
+			_log.warn("No properties found for " + _componentName)
+		}
+
 		if (!componentProperties.hasBaseConfiguration()) {
 			throw new ConfigurationNotFoundException(
 				_componentName, "The base properties file was not found");

liferay-plugins-sdk-6.2/portlets/portos-bo-portlet/docroot/WEB-INF/service/it/tref/liferay/portos/bo/util/MailUtil.java

@@ -161,7 +161,9 @@ public class MailUtil {
 			intPraticaId = parereGeologo.getIntPraticaId();
 			IntPratica intPratica = IntPraticaLocalServiceUtil.getIntPratica(intPraticaId);
 			User tecnicoIstruttore = recuperaTecnicoIstruttore(intPratica);
+			if(tecnicoIstruttore != null){
 				_log.info("email tecnico istruttore=" + tecnicoIstruttore.getEmailAddress());
+				
 				String[] to = { tecnicoIstruttore.getEmailAddress() };
 				String[] cc = new String[0];
 				String[] ccn = new String[0];
@@ -192,6 +194,10 @@ public class MailUtil {
 				long classPk = parereGeologoId;
 				MailManagerUtil.sendMailByTemplatName(className, classPk, "NOTIFICA-BO", to, cc, ccn, templateVariables,
 						null, serviceContext);
+			} else {
+				_log.warn("invioNotificaIstruttoreByGeologo - Tecnico istruttore mancante per pratica id="
+						+ intPraticaId);
+			}
 		} catch (Exception e) {
 			_log.error("invioNotificaIstruttoreByGeologo - Errore invio mail al tecnico istruttore per pratica id="
 					+ intPraticaId, e);
@@ -248,9 +254,11 @@ public class MailUtil {
 		User istruttoreUser = null;
 		if (Validator.isNotNull(statusByUserId) && intPratica.getStatus() != WorkflowConstants.STATUS_DRAFT) {
 			istruttoreUser = UserLocalServiceUtil.getUser(statusByUserId);
+			if(istruttoreUser != null){
 				_log.info("istruttore pratica id=" + intPratica.getIntPraticaId() + ":" + istruttoreUser.getScreenName()
 						+ " " + istruttoreUser.getEmailAddress());
 			}
+		}
 		return istruttoreUser;
 	}
 

liferay-plugins-sdk-6.2/portlets/portos-bo-portlet/docroot/WEB-INF/service/it/tref/liferay/portos/bo/util/WorkflowUtil.java

@@ -256,10 +256,12 @@ public class WorkflowUtil {
 			ControlloPratica controlloPratica) throws PortalException, SystemException {
 
 		WorkflowTask workflowTask = getLastTask(companyId, groupId, null, controlloPratica);
+		if(workflowTask != null) {
 			Map<String, Serializable> workflowContext = getWorkflowContext(controlloPratica);
 			WorkflowTaskManagerUtil.assignWorkflowTaskToUser(companyId, assegnatoreUserId,
 					workflowTask.getWorkflowTaskId(), istruttoreId, StringPool.BLANK, null, workflowContext);
 		}
+	}
 
 	public static Map<String, Serializable> getWorkflowContext(ControlloPratica controlloPratica)
 			throws PortalException, SystemException {

liferay-plugins-sdk-6.2/portlets/portos-bo-portlet/docroot/WEB-INF/src/it/mwg/sismica/bo/util/ManutenzioneDatabase.java

@@ -1089,7 +1089,9 @@ public class ManutenzioneDatabase {
 					try {
 						instance = WorkflowInstanceManagerUtil.getWorkflowInstance(link.getCompanyId(),
 								link.getWorkflowInstanceId());
-					} catch (PortalException e) {}
+					} catch (PortalException e) {
+						_log.error(e, e);
+					}
 					if (Validator.isNull(instance)) {
 						WorkflowInstanceLinkLocalServiceUtil.deleteWorkflowInstanceLink(link);
 					} else {

liferay-plugins-sdk-6.2/portlets/portos-bo-portlet/docroot/WEB-INF/src/it/mwg/sismica/bo/util/SismicaUtil.java

@@ -726,6 +726,7 @@ public class SismicaUtil {
 										+ pratica.getNumeroProgetto() + ": " + e2.getMessage());
 							}
 						}
+						if(controllo != null){
 							controllo.setStatus(WorkflowConstants.STATUS_APPROVED);
 							AvvisoLocalServiceUtil.addAvviso(pratica.getIntPraticaId(),
 									"Attestazione di deposito progetto", new Date(), AvvisoUtil.TIPO_AVVISO_DIRETTO,
@@ -735,6 +736,7 @@ public class SismicaUtil {
 							IntPraticaLocalServiceUtil.updateIntPratica(pratica);
 						}
 					}
+				}
 			} catch (PortalException | SystemException e) {
 				esitoLog(esito, "Errore durante l'elaborazione: " + e.getMessage());
 			}
@@ -958,7 +960,7 @@ public class SismicaUtil {
 						if (tokens.contains(controllo.getControlloPraticaId()))
 							tokenCount++;
 					}
-					if (0 == tokenCount) {
+					if (0 == tokenCount && Validator.isNotNull(last)) {
 						ServiceContext serviceContext = new ServiceContext();
 						serviceContext.setCompanyId(last.getCompanyId());
 						serviceContext.setScopeGroupId(last.getGroupId());
@@ -1266,7 +1268,7 @@ public class SismicaUtil {
 
 	public static List<String> resetFirme(long companyId, String numeroProgetto) throws SystemException,
 			PortalException {
-
+		try{
 			List<String> esito = new ArrayList<>();
 			if (Validator.isNull(numeroProgetto)) {
 				esito.addAll(firmeBloccate(companyId));
@@ -1314,6 +1316,9 @@ public class SismicaUtil {
 					}
 				}
 			}
+		}catch(Exception e){
+			_log.error(e, e);
+		}
 		return esito;
 	}
 
@@ -1452,20 +1457,24 @@ public class SismicaUtil {
 				IntPratica pratica = null;
 				try {
 					pratica = IntPraticaLocalServiceUtil.getIntPratica(documento.getIntPraticaId());
-				} catch (PortalException e) {}
+				} catch (PortalException e) {
+					_log.error(e,e);
+				}
 				if (Validator.isNotNull(pratica) && (documento.getFileEntryId() != 0)) {
 					Territorio territorio = TerritorioLocalServiceUtil.getTerritorio(pratica.getTerritorioId());
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
 					} catch (Exception e) {
-						// _log.info(e.getClass() + ": " + e.getMessage());
+						_log.error(e,e);
 					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1499,12 +1508,16 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
-					} catch (Exception e) {}
+					} catch (Exception e) {
+						_log.error(e,e);
+					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1538,10 +1551,14 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryIdBase());
-					} catch (Exception e) {}
+					} catch (Exception e) {
+						_log.error(e,e);
+					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
+					}
 
 				}
 				if (Validator.isNotNull(pratica) && (documento.getFileEntryIdInvio() != 0)) {
@@ -1549,22 +1566,30 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryIdInvio());
-					} catch (Exception e) {}
+					} catch (Exception e) {
+						_log.error(e,e);
+					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
+				}
 				if (Validator.isNotNull(pratica) && (documento.getFileEntryIdProtocollo() != 0)) {
 					Territorio territorio = TerritorioLocalServiceUtil.getTerritorio(pratica.getTerritorioId());
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryIdProtocollo());
-					} catch (Exception e) {}
+					} catch (Exception e) {
+						_log.error(e,e);
+					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1599,13 +1624,15 @@ public class SismicaUtil {
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
 					} catch (Exception e) {
-						// _log.info(e.getClass() + ": " + e.getMessage());
+						_log.error(e,e);
 					}
+					if(Validator.isNotNull(fileEntry){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1640,13 +1667,15 @@ public class SismicaUtil {
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getDlFileEntryId());
 					} catch (Exception e) {
-						// _log.info(e.getClass() + ": " + e.getMessage());
+						_log.error(e,e);
 					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1681,13 +1710,15 @@ public class SismicaUtil {
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
 					} catch (Exception e) {
-						// _log.info(e.getClass() + ": " + e.getMessage());
+						_log.error(e,e);
 					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();
@@ -1722,13 +1753,15 @@ public class SismicaUtil {
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
 					} catch (Exception e) {
-						// _log.info(e.getClass() + ": " + e.getMessage());
+						_log.error(e,e);
 					}
+					if(Validator.isNotNull(fileEntry)){
 						corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
 								+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
 								+ StringPool.SPACE + documento.getPrimaryKey());
 					}
 				}
+			}
 			inizio += SLICE;
 			if (documenti.size() > 0) {
 				totali += documenti.size();