shuber 2004/09/24 18:36:42 CEST
Modified files:
core/src/java/org/jahia/services/fields ContentPageField.java
Log:
Modified warnings in activation to be result objects instead of Strings so that we
generate the proper warning displays in the workflow page, showing links to edition
popups.
Revision Changes Path
1.5 +65 -14
jahia/core/src/java/org/jahia/services/fields/ContentPageField.java
http://jahia.mine.nu:8080/cgi-bin/cvsweb.cgi/jahia/core/src/java/org/jahia/services/fields/ContentPageField.java.diff?r1=1.4&r2=1.5&f=h
Index: ContentPageField.java
===================================================================
RCS file: /cvs/jahia/core/src/java/org/jahia/services/fields/ContentPageField.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- ContentPageField.java 19 Aug 2004 16:19:08 -0000 1.4
+++ ContentPageField.java 24 Sep 2004 16:36:42 -0000 1.5
@@ -298,17 +298,34 @@
if (
!stateModifContext.getStartObject().equals(contentPage.getObjectKey()) ){
activationTestResults.setStatus(ActivationTestResults.
FAILED_OPERATION_STATUS);
- activationTestResults.appendWarning("Field " + getID() +
- " reffered to a moved page");
+ try {
+ IsValidForActivationResults activationResults = new
+ IsValidForActivationResults ( ContentFieldKey.FIELD_TYPE,
+ getID (),
+ jParams.getLocale ().toString
(),
+ "Field " + getID() +
+ " referred to a moved page");
+ activationTestResults.appendWarning(activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
return activationTestResults;
}
}
if (contentPage == null) {
// this could happen if the database had a jahia_link_only value
activationTestResults.setStatus
(ActivationTestResults.PARTIAL_OPERATION_STATUS);
- activationTestResults.appendWarning (
- "Field " + getID () + " value is not correctly set. Current
value=" + getValue (
- jParams));
+ try {
+ IsValidForActivationResults activationResults = new
+ IsValidForActivationResults (ContentFieldKey.FIELD_TYPE,
+ getID (),
+ jParams.getLocale ().toString (),
+ "Field " + getID () + " value is
not set. Current value=" +
+ getValue (jParams));
+ activationTestResults.appendWarning (activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
return activationTestResults;
}
@@ -327,7 +344,16 @@
if (thePage == null) {
activationTestResults.setStatus
(ActivationTestResults.FAILED_OPERATION_STATUS);
- activationTestResults.appendWarning ("Could not lookup page " +
pageID + " !");
+ try {
+ IsValidForActivationResults activationResults = new
+ IsValidForActivationResults (ContentFieldKey.FIELD_TYPE,
+ getID (),
+ jParams.getLocale ().toString
(),
+ "Could not lookup page " +
pageID + " !");
+ activationTestResults.appendWarning (activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
return activationTestResults;
}
@@ -357,8 +383,18 @@
} else {
activationTestResults.setStatus (
ActivationTestResults.FAILED_OPERATION_STATUS);
- activationTestResults.appendWarning (
- "Page " + pageID + " only has staged entries
and recursive activation is not activated.");
+ try {
+ IsValidForActivationResults activationResults = new
+ IsValidForActivationResults
(ContentFieldKey.FIELD_TYPE,
+ getID (),
+ jParams.getLocale
().toString (),
+ "Page " +
+ pageID +
+ " exists only in
staging and needs to be validated.");
+ activationTestResults.appendWarning
(activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
}
break;
@@ -379,22 +415,37 @@
ActivationTestResults.
COMPLETED_OPERATION_STATUS);
} else {
- activationTestResults.setStatus (
- ActivationTestResults.
- PARTIAL_OPERATION_STATUS);
- activationTestResults.appendWarning (
+ activationTestResults.setStatus
(ActivationTestResults.PARTIAL_OPERATION_STATUS);
+ try {
+ IsValidForActivationResults activationResults =
new
+ IsValidForActivationResults (
ContentFieldKey.FIELD_TYPE,
+ getID (),
+
jParams.getLocale ().toString (),
"Page " +
pageID +
- " only has staged entries and recursive
activation is not activated.");
+ " exists only
in staging and needs to be validated.");
+ activationTestResults.appendWarning
(activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
}
} else {
activationTestResults.setStatus (
ActivationTestResults.
PARTIAL_OPERATION_STATUS);
- activationTestResults.appendWarning ("Page " +
+ try {
+ IsValidForActivationResults activationResults = new
+ IsValidForActivationResults (
ContentFieldKey.FIELD_TYPE,
+ getID (),
+ jParams.getLocale
().toString (),
+ "Page " +
pageID +
" has a link to a page that couldn't be found :
" +
thePage.getPageLinkID () + " !");
+ activationTestResults.appendWarning (activationResults);
+ } catch (ClassNotFoundException cnfe) {
+ logger.error(cnfe);
+ }
}
break;
