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,37 +161,43 @@ public class MailUtil {
 			intPraticaId = parereGeologo.getIntPraticaId();
 			IntPratica intPratica = IntPraticaLocalServiceUtil.getIntPratica(intPraticaId);
 			User tecnicoIstruttore = recuperaTecnicoIstruttore(intPratica);
-			_log.info("email tecnico istruttore=" + tecnicoIstruttore.getEmailAddress());
-			String[] to = { tecnicoIstruttore.getEmailAddress() };
-			String[] cc = new String[0];
-			String[] ccn = new String[0];
-			// recupero dati
-			String tipoPratica = StringPool.BLANK;
-			if (AzioniPraticheUtil.isDeposito(intPratica.getIntPraticaId())) {
-				tipoPratica = LanguageUtil.get(LocaleUtil.ITALIAN, "tipo-pratica-01");
-			} else if (AzioniPraticheUtil.isAutorizzazione(intPratica.getIntPraticaId())) {
-				tipoPratica = LanguageUtil.get(LocaleUtil.ITALIAN, "tipo-pratica-02");
+			if(tecnicoIstruttore != null){
+				_log.info("email tecnico istruttore=" + tecnicoIstruttore.getEmailAddress());
+				
+				String[] to = { tecnicoIstruttore.getEmailAddress() };
+				String[] cc = new String[0];
+				String[] ccn = new String[0];
+				// recupero dati
+				String tipoPratica = StringPool.BLANK;
+				if (AzioniPraticheUtil.isDeposito(intPratica.getIntPraticaId())) {
+					tipoPratica = LanguageUtil.get(LocaleUtil.ITALIAN, "tipo-pratica-01");
+				} else if (AzioniPraticheUtil.isAutorizzazione(intPratica.getIntPraticaId())) {
+					tipoPratica = LanguageUtil.get(LocaleUtil.ITALIAN, "tipo-pratica-02");
+				}
+				Territorio territorio = TerritorioLocalServiceUtil.getTerritorio(intPratica.getTerritorioId());
+				Comune comune = ComuneLocalServiceUtil.getComune(territorio.getComuneId());
+				Provincia provincia = ProvinciaLocalServiceUtil.fetchByC_C(territorio.getCompanyId(),
+						territorio.getCodiceProvincia());
+				JSONObject templateVariables = JSONFactoryUtil.createJSONObject();
+				templateVariables.put("receiver", tecnicoIstruttore.getFullName());
+				templateVariables.put("tipoPratica", tipoPratica);
+				templateVariables.put("tipoOggetto", "Risposta a Richiesta Parere Geologo");
+				templateVariables
+						.put("dataInvio",
+								FastDateFormatFactoryUtil.getSimpleDateFormat("dd/MM/yyyy HH:mm").format(
+										parereGeologo.getDtCompilazione() != null ? parereGeologo.getDtCompilazione()
+												: new Date()));
+				templateVariables.put("praticaNum", intPratica.getNumeroProgetto());
+				templateVariables.put("provincia", provincia.getProvincia());
+				templateVariables.put("comune", comune.getDenominazione());
+				String className = ParereGeologo.class.getName();
+				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);
 			}
-			Territorio territorio = TerritorioLocalServiceUtil.getTerritorio(intPratica.getTerritorioId());
-			Comune comune = ComuneLocalServiceUtil.getComune(territorio.getComuneId());
-			Provincia provincia = ProvinciaLocalServiceUtil.fetchByC_C(territorio.getCompanyId(),
-					territorio.getCodiceProvincia());
-			JSONObject templateVariables = JSONFactoryUtil.createJSONObject();
-			templateVariables.put("receiver", tecnicoIstruttore.getFullName());
-			templateVariables.put("tipoPratica", tipoPratica);
-			templateVariables.put("tipoOggetto", "Risposta a Richiesta Parere Geologo");
-			templateVariables
-					.put("dataInvio",
-							FastDateFormatFactoryUtil.getSimpleDateFormat("dd/MM/yyyy HH:mm").format(
-									parereGeologo.getDtCompilazione() != null ? parereGeologo.getDtCompilazione()
-											: new Date()));
-			templateVariables.put("praticaNum", intPratica.getNumeroProgetto());
-			templateVariables.put("provincia", provincia.getProvincia());
-			templateVariables.put("comune", comune.getDenominazione());
-			String className = ParereGeologo.class.getName();
-			long classPk = parereGeologoId;
-			MailManagerUtil.sendMailByTemplatName(className, classPk, "NOTIFICA-BO", to, cc, ccn, templateVariables,
-					null, serviceContext);
 		} catch (Exception e) {
 			_log.error("invioNotificaIstruttoreByGeologo - Errore invio mail al tecnico istruttore per pratica id="
 					+ intPraticaId, e);
@@ -248,8 +254,10 @@ public class MailUtil {
 		User istruttoreUser = null;
 		if (Validator.isNotNull(statusByUserId) && intPratica.getStatus() != WorkflowConstants.STATUS_DRAFT) {
 			istruttoreUser = UserLocalServiceUtil.getUser(statusByUserId);
-			_log.info("istruttore pratica id=" + intPratica.getIntPraticaId() + ":" + istruttoreUser.getScreenName()
-					+ " " + istruttoreUser.getEmailAddress());
+			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,9 +256,11 @@ public class WorkflowUtil {
 			ControlloPratica controlloPratica) throws PortalException, SystemException {
 
 		WorkflowTask workflowTask = getLastTask(companyId, groupId, null, controlloPratica);
-		Map<String, Serializable> workflowContext = getWorkflowContext(controlloPratica);
-		WorkflowTaskManagerUtil.assignWorkflowTaskToUser(companyId, assegnatoreUserId,
-				workflowTask.getWorkflowTaskId(), istruttoreId, StringPool.BLANK, null, workflowContext);
+		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)

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,13 +726,15 @@ public class SismicaUtil {
 										+ pratica.getNumeroProgetto() + ": " + e2.getMessage());
 							}
 						}
-						controllo.setStatus(WorkflowConstants.STATUS_APPROVED);
-						AvvisoLocalServiceUtil.addAvviso(pratica.getIntPraticaId(),
-								"Attestazione di deposito progetto", new Date(), AvvisoUtil.TIPO_AVVISO_DIRETTO,
-								StatoPraticaConstants.CONFORME, fileEntryIdTemplate, dettPratica.getDettPraticaId(),
-								DettPratica.class.getName(), controllo.getControlloPraticaId(), serviceContext);
-						pratica.setStatoPratica(StatoPraticaConstants.CONFORME);
-						IntPraticaLocalServiceUtil.updateIntPratica(pratica);
+						if(controllo != null){
+							controllo.setStatus(WorkflowConstants.STATUS_APPROVED);
+							AvvisoLocalServiceUtil.addAvviso(pratica.getIntPraticaId(),
+									"Attestazione di deposito progetto", new Date(), AvvisoUtil.TIPO_AVVISO_DIRETTO,
+									StatoPraticaConstants.CONFORME, fileEntryIdTemplate, dettPratica.getDettPraticaId(),
+									DettPratica.class.getName(), controllo.getControlloPraticaId(), serviceContext);
+							pratica.setStatoPratica(StatoPraticaConstants.CONFORME);
+							IntPraticaLocalServiceUtil.updateIntPratica(pratica);
+						}
 					}
 				}
 			} catch (PortalException | SystemException e) {
@@ -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,53 +1268,56 @@ public class SismicaUtil {
 
 	public static List<String> resetFirme(long companyId, String numeroProgetto) throws SystemException,
 			PortalException {
-
-		List<String> esito = new ArrayList<>();
-		if (Validator.isNull(numeroProgetto)) {
-			esito.addAll(firmeBloccate(companyId));
-		} else {
-			IntPratica pratica = IntPraticaLocalServiceUtil.findByNumeroProgetto(numeroProgetto);
-			ControlloPratica controllo = getLastControlloPratica(pratica.getIntPraticaId());
-			KaleoInstanceToken instanceToken = findInstanceToken(controllo.getControlloPraticaId());
-			if (Validator.isNotNull(instanceToken)) {
-				esitoLog(esito, "Trovato KaleoInstanceToken " + instanceToken.getKaleoInstanceTokenId()
-						+ ", currentKaleoNodeName = \"" + instanceToken.getCurrentKaleoNodeName() + StringPool.QUOTE);
-				if (instanceToken.getCurrentKaleoNodeName().equalsIgnoreCase("signal_ff")) {
-					List<KaleoTaskInstanceToken> taskInstanceTokens = findTaskInstanceTokens(instanceToken
-							.getKaleoInstanceTokenId());
-					esitoLog(esito, "Trovati " + taskInstanceTokens.size() + " KaleoTaskInstanceToken");
-					List<KaleoTaskAssignmentInstance> taskAssignmentInstances = findTaskAssignmentInstances(instanceToken
-							.getKaleoInstanceId());
-					esitoLog(esito, "Trovati " + taskAssignmentInstances.size() + " KaleoTaskAssignmentInstance");
-
-					KaleoTaskInstanceToken taskInstanceToken = taskInstanceTokens.get(taskInstanceTokens.size() - 1);
-					KaleoTaskAssignmentInstance taskAssignmentInstance = taskAssignmentInstances
-							.get(taskAssignmentInstances.size() - 1);
-
-					esitoLog(esito,
-							"Cancello KaleoTaskInstanceToken " + taskInstanceToken.getKaleoTaskInstanceTokenId());
-					KaleoTaskInstanceTokenLocalServiceUtil.deleteKaleoTaskInstanceToken(taskInstanceToken);
-					esitoLog(
-							esito,
-							"Cancello KaleoTaskAssignmentInstance "
-									+ taskAssignmentInstance.getKaleoTaskAssignmentInstanceId());
-					KaleoTaskAssignmentInstanceLocalServiceUtil
-							.deleteKaleoTaskAssignmentInstance(taskAssignmentInstance);
-
-					taskInstanceToken = taskInstanceTokens.get(taskInstanceTokens.size() - 2);
-					esitoLog(esito,
-							"Aggiorno KaleoTaskInstanceToken " + taskInstanceToken.getKaleoTaskInstanceTokenId());
-					taskInstanceToken.setCompleted(false);
-					taskInstanceToken.setCompletionUserId(0);
-					taskInstanceToken.setCompletionDate(null);
-					KaleoTaskInstanceTokenLocalServiceUtil.updateKaleoTaskInstanceToken(taskInstanceToken);
-
-					esitoLog(esito, "Aggiorno KaleoInstanceToken " + instanceToken.getKaleoInstanceTokenId());
-					instanceToken.setCurrentKaleoNodeId(21639);
-					instanceToken.setCurrentKaleoNodeName("Esito");
-					KaleoInstanceTokenLocalServiceUtil.updateKaleoInstanceToken(instanceToken);
+		try{
+			List<String> esito = new ArrayList<>();
+			if (Validator.isNull(numeroProgetto)) {
+				esito.addAll(firmeBloccate(companyId));
+			} else {
+				IntPratica pratica = IntPraticaLocalServiceUtil.findByNumeroProgetto(numeroProgetto);
+				ControlloPratica controllo = getLastControlloPratica(pratica.getIntPraticaId());
+				KaleoInstanceToken instanceToken = findInstanceToken(controllo.getControlloPraticaId());
+				if (Validator.isNotNull(instanceToken)) {
+					esitoLog(esito, "Trovato KaleoInstanceToken " + instanceToken.getKaleoInstanceTokenId()
+							+ ", currentKaleoNodeName = \"" + instanceToken.getCurrentKaleoNodeName() + StringPool.QUOTE);
+					if (instanceToken.getCurrentKaleoNodeName().equalsIgnoreCase("signal_ff")) {
+						List<KaleoTaskInstanceToken> taskInstanceTokens = findTaskInstanceTokens(instanceToken
+								.getKaleoInstanceTokenId());
+						esitoLog(esito, "Trovati " + taskInstanceTokens.size() + " KaleoTaskInstanceToken");
+						List<KaleoTaskAssignmentInstance> taskAssignmentInstances = findTaskAssignmentInstances(instanceToken
+								.getKaleoInstanceId());
+						esitoLog(esito, "Trovati " + taskAssignmentInstances.size() + " KaleoTaskAssignmentInstance");
+
+						KaleoTaskInstanceToken taskInstanceToken = taskInstanceTokens.get(taskInstanceTokens.size() - 1);
+						KaleoTaskAssignmentInstance taskAssignmentInstance = taskAssignmentInstances
+								.get(taskAssignmentInstances.size() - 1);
+
+						esitoLog(esito,
+								"Cancello KaleoTaskInstanceToken " + taskInstanceToken.getKaleoTaskInstanceTokenId());
+						KaleoTaskInstanceTokenLocalServiceUtil.deleteKaleoTaskInstanceToken(taskInstanceToken);
+						esitoLog(
+								esito,
+								"Cancello KaleoTaskAssignmentInstance "
+										+ taskAssignmentInstance.getKaleoTaskAssignmentInstanceId());
+						KaleoTaskAssignmentInstanceLocalServiceUtil
+								.deleteKaleoTaskAssignmentInstance(taskAssignmentInstance);
+
+						taskInstanceToken = taskInstanceTokens.get(taskInstanceTokens.size() - 2);
+						esitoLog(esito,
+								"Aggiorno KaleoTaskInstanceToken " + taskInstanceToken.getKaleoTaskInstanceTokenId());
+						taskInstanceToken.setCompleted(false);
+						taskInstanceToken.setCompletionUserId(0);
+						taskInstanceToken.setCompletionDate(null);
+						KaleoTaskInstanceTokenLocalServiceUtil.updateKaleoTaskInstanceToken(taskInstanceToken);
+
+						esitoLog(esito, "Aggiorno KaleoInstanceToken " + instanceToken.getKaleoInstanceTokenId());
+						instanceToken.setCurrentKaleoNodeId(21639);
+						instanceToken.setCurrentKaleoNodeName("Esito");
+						KaleoInstanceTokenLocalServiceUtil.updateKaleoInstanceToken(instanceToken);
+					}
 				}
 			}
+		}catch(Exception e){
+			_log.error(e, e);
 		}
 		return esito;
 	}
@@ -1452,18 +1457,22 @@ 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());
 					}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
 				}
 			}
 			inizio += SLICE;
@@ -1499,10 +1508,14 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryId());
-					} catch (Exception e) {}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
+					} 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;
@@ -1538,10 +1551,14 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryIdBase());
-					} catch (Exception e) {}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
+					} 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,20 +1566,28 @@ public class SismicaUtil {
 					FileEntry fileEntry = null;
 					try {
 						fileEntry = DLAppServiceUtil.getFileEntry(documento.getFileEntryIdInvio());
-					} catch (Exception e) {}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
+					} 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) {}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
+					} 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;
@@ -1599,11 +1624,13 @@ 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());
 					}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
 				}
 			}
 			inizio += SLICE;
@@ -1640,11 +1667,13 @@ 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());
 					}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
 				}
 			}
 			inizio += SLICE;
@@ -1681,11 +1710,13 @@ 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());
 					}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
 				}
 			}
 			inizio += SLICE;
@@ -1722,11 +1753,13 @@ 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());
 					}
-					corretti += riparaPermessiFileEntry(companyId, territorio.getGroupId(), fileEntry, "Pratica "
-							+ pratica.getNumeroProgetto() + " (" + pratica.getIntPraticaId() + "), " + tipoDocumento
-							+ StringPool.SPACE + documento.getPrimaryKey());
 				}
 			}
 			inizio += SLICE;