slub.team team mailing list archive

[Henning Gerhardt <henning.gerhardt@xxxxxxxxxxxxxxx>]

Henning Gerhardt has proposed merging lp:~henning-gerhardt/goobi-production/bug-904716 into lp:goobi-production.

Requested reviews:
  Ralf Claussnitzer (ralf-claussnitzer)
Related bugs:
  Bug #904716 in Goobi.Production: "application crashes on adding process step property"
  https://bugs.launchpad.net/goobi-production/+bug/904716

For more details, see:
https://code.launchpad.net/~henning-gerhardt/goobi-production/bug-904716/+merge/96108

CRUD operation for properties (Prozess, Vorlage, Schritte, Werkstücke) should work again.
-- 
https://code.launchpad.net/~henning-gerhardt/goobi-production/bug-904716/+merge/96108
Your team Saxon State Library Team is subscribed to branch lp:goobi-production.

=== modified file 'newpages/inc_Prozessverwaltung/prozess_box_Prozesseigenschaften.jsp'
--- newpages/inc_Prozessverwaltung/prozess_box_Prozesseigenschaften.jsp	2011-12-20 08:07:09 +0000
+++ newpages/inc_Prozessverwaltung/prozess_box_Prozesseigenschaften.jsp	2012-03-06 12:25:21 +0000
@@ -39,7 +39,7 @@
 	cellspacing="1px" cellpadding="1px" headerClass="standardTable_Header"
 	rowClasses="standardTable_Row1"
 	columnClasses="standardTable_Column,standardTable_Column,standardTable_ColumnCentered"
-	var="item" value="#{ProzessverwaltungForm.myProzess.eigenschaftenList}"
+	var="item" value="#{ProzessverwaltungForm.myProzess.displayProperties.sortedProperties}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='eigenschaft'}">
 
 	<h:column>
@@ -66,7 +66,7 @@
 			title="#{msgs.eigenschaftBearbeiten}">
 			<h:graphicImage value="/newpages/images/buttons/edit.gif" />
 			<x:updateActionListener
-				property="#{ProzessverwaltungForm.myProzessEigenschaft}"
+				property="#{ProzessverwaltungForm.myProzess.displayProperties.currentProperty}"
 				value="#{item}" />
 			<x:updateActionListener
 				property="#{ProzessverwaltungForm.modusBearbeiten}"
@@ -76,7 +76,7 @@
 </x:dataTable>
 
 <%-- Neu-Schaltknopf --%>
-<h:commandLink action="#{ProzessverwaltungForm.ProzessEigenschaftNeu}"
+<h:commandLink action="#{ProzessverwaltungForm.myProzess.displayProperties.createNewProperty}"
 	value="#{msgs.eigenschaftHinzufuegen}"
 	title="#{msgs.eigenschaftHinzufuegen}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='eigenschaft' && ((LoginForm.maximaleBerechtigung == 1) || (LoginForm.maximaleBerechtigung == 2))}">
@@ -116,7 +116,7 @@
 					<h:panelGroup>
 						<h:inputText id="eigenschafttitel"
 							style="width: 300px;margin-right:15px"
-							value="#{ProzessverwaltungForm.myProzessEigenschaft.titel}"
+							value="#{ProzessverwaltungForm.myProzess.displayProperties.currentProperty.titel}"
 							required="true" />
 						<x:message for="eigenschafttitel" style="color: red"
 							detailFormat="#{msgs.keinTitelAngegeben}" />
@@ -126,7 +126,7 @@
 					<h:panelGroup>
 						<h:inputText id="eigenschaftwert"
 							style="width: 300px;margin-right:15px"
-							value="#{ProzessverwaltungForm.myProzessEigenschaft.wert}" />
+							value="#{ProzessverwaltungForm.myProzess.displayProperties.currentProperty.wert}" />
 					</h:panelGroup>
 				</h:panelGrid>
 
@@ -143,9 +143,9 @@
 			</htm:td>
 			<htm:td styleClass="eingabeBoxen_row3" align="right">
 				<h:commandButton value="#{msgs.loeschen}"
-					action="#{ProzessverwaltungForm.ProzessEigenschaftLoeschen}"
+					action="#{ProzessverwaltungForm.myProzess.displayProperties.deleteProperty}"
 					onclick="return confirm('#{msgs.sollDieserEintragWirklichGeloeschtWerden}?')"
-					rendered="#{ProzessverwaltungForm.myProzessEigenschaft.id != null}">
+					rendered="#{ProzessverwaltungForm.myProzess.displayProperties.currentProperty.id != null}">
 					<x:updateActionListener
 						property="#{ProzessverwaltungForm.modusBearbeiten}" value="" />
 				</h:commandButton>

=== modified file 'newpages/inc_Prozessverwaltung/schritt_box_Eigenschaften.jsp'
--- newpages/inc_Prozessverwaltung/schritt_box_Eigenschaften.jsp	2011-12-20 08:07:09 +0000
+++ newpages/inc_Prozessverwaltung/schritt_box_Eigenschaften.jsp	2012-03-06 12:25:21 +0000
@@ -39,7 +39,7 @@
 	cellspacing="1px" cellpadding="1px" headerClass="standardTable_Header"
 	rowClasses="standardTable_Row1"
 	columnClasses="standardTable_Column,standardTable_Column,standardTable_ColumnCentered"
-	var="item" value="#{ProzessverwaltungForm.mySchritt.eigenschaftenList}"
+	var="item" value="#{ProzessverwaltungForm.mySchritt.displayProperties.sortedProperties}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='schritteeigenschaft'}">
 
 	<h:column>
@@ -65,7 +65,7 @@
 			title="#{msgs.bearbeiten}">
 			<h:graphicImage value="/newpages/images/buttons/edit.gif" />
 			<x:updateActionListener
-				property="#{ProzessverwaltungForm.mySchrittEigenschaft}"
+				property="#{ProzessverwaltungForm.mySchritt.displayProperties.currentProperty}"
 				value="#{item}" />
 			<x:updateActionListener
 				property="#{ProzessverwaltungForm.modusBearbeiten}"
@@ -75,7 +75,7 @@
 </x:dataTable>
 
 <%-- Neu-Schaltknopf --%>
-<h:commandLink action="#{ProzessverwaltungForm.SchrittEigenschaftNeu}"
+<h:commandLink action="#{ProzessverwaltungForm.mySchritt.displayProperties.createNewProperty}"
 	value="#{msgs.eigenschaftHinzufuegen}"
 	title="#{msgs.eigenschaftHinzufuegen}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='schritteeigenschaft'}">
@@ -114,7 +114,7 @@
 				<h:panelGroup>
 					<h:inputText id="eigenschafttitel"
 						style="width: 300px;margin-right:15px"
-						value="#{ProzessverwaltungForm.mySchrittEigenschaft.titel}"
+						value="#{ProzessverwaltungForm.mySchritt.displayProperties.currentProperty.titel}"
 						required="true" />
 					<x:message for="eigenschafttitel" style="color: red"
 						detailFormat="#{msgs.keinTitelAngegeben}" />
@@ -124,7 +124,7 @@
 				<h:panelGroup>
 					<h:inputText id="eigenschaftwert"
 						style="width: 300px;margin-right:15px"
-						value="#{ProzessverwaltungForm.mySchrittEigenschaft.wert}" />
+						value="#{ProzessverwaltungForm.mySchritt.displayProperties.currentProperty.wert}" />
 				</h:panelGroup>
 			</h:panelGrid>
 
@@ -141,9 +141,9 @@
 		</htm:td>
 		<htm:td styleClass="eingabeBoxen_row3" align="right">
 			<h:commandButton value="#{msgs.loeschen}"
-				action="#{ProzessverwaltungForm.SchrittEigenschaftLoeschen}"
+				action="#{ProzessverwaltungForm.mySchritt.displayProperties.deleteProperty}"
 				onclick="return confirm('#{msgs.sollDieserEintragWirklichGeloeschtWerden}?')"
-				rendered="#{ProzessverwaltungForm.mySchrittEigenschaft.id != null}">
+				rendered="#{ProzessverwaltungForm.mySchritt.displayProperties.currentProperty.id != null}">
 				<x:updateActionListener
 					property="#{ProzessverwaltungForm.modusBearbeiten}" value="" />
 			</h:commandButton>

=== modified file 'newpages/inc_Prozessverwaltung/vorlage_box_Eigenschaften.jsp'
--- newpages/inc_Prozessverwaltung/vorlage_box_Eigenschaften.jsp	2011-12-20 08:07:09 +0000
+++ newpages/inc_Prozessverwaltung/vorlage_box_Eigenschaften.jsp	2012-03-06 12:25:21 +0000
@@ -39,7 +39,7 @@
 	width="100%" cellspacing="1px" cellpadding="1px"
 	headerClass="standardTable_Header" rowClasses="standardTable_Row1"
 	columnClasses="standardTable_Column,standardTable_Column,standardTable_ColumnCentered"
-	var="item" value="#{ProzessverwaltungForm.myVorlage.eigenschaftenList}"
+	var="item" value="#{ProzessverwaltungForm.myVorlage.displayProperties.sortedProperties}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='vorlageeigenschaft'}">
 
 	<h:column>
@@ -65,7 +65,7 @@
 			title="#{msgs.bearbeiten}">
 			<h:graphicImage value="/newpages/images/buttons/edit.gif" />
 			<x:updateActionListener
-				property="#{ProzessverwaltungForm.myVorlageEigenschaft}"
+				property="#{ProzessverwaltungForm.myVorlage.displayProperties.currentProperty}"
 				value="#{item}" />
 			<x:updateActionListener
 				property="#{ProzessverwaltungForm.modusBearbeiten}"
@@ -75,7 +75,7 @@
 </x:dataTable>
 
 <%-- Neu-Schaltknopf --%>
-<h:commandLink action="#{ProzessverwaltungForm.VorlageEigenschaftNeu}"
+<h:commandLink action="#{ProzessverwaltungForm.myVorlage.displayProperties.createNewProperty}"
 	value="#{msgs.eigenschaftHinzufuegen}"
 	title="#{msgs.eigenschaftHinzufuegen}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='vorlageeigenschaft'}">
@@ -114,7 +114,7 @@
 				<h:panelGroup>
 					<h:inputText id="eigenschafttitel"
 						style="width: 300px;margin-right:15px"
-						value="#{ProzessverwaltungForm.myVorlageEigenschaft.titel}"
+						value="#{ProzessverwaltungForm.myVorlage.displayProperties.currentProperty.titel}"
 						required="true" />
 					<x:message for="eigenschafttitel" style="color: red"
 						detailFormat="#{msgs.keinTitelAngegeben}" />
@@ -124,7 +124,7 @@
 				<h:panelGroup>
 					<h:inputText id="eigenschaftwert"
 						style="width: 300px;margin-right:15px"
-						value="#{ProzessverwaltungForm.myVorlageEigenschaft.wert}" />
+						value="#{ProzessverwaltungForm.myVorlage.displayProperties.currentProperty.wert}" />
 				</h:panelGroup>
 			</h:panelGrid>
 
@@ -141,9 +141,9 @@
 		</htm:td>
 		<htm:td styleClass="eingabeBoxen_row3" align="right">
 			<h:commandButton value="#{msgs.loeschen}"
-				action="#{ProzessverwaltungForm.VorlageEigenschaftLoeschen}"
+				action="#{ProzessverwaltungForm.myVorlage.displayProperties.deleteProperty}"
 				onclick="return confirm('#{msgs.sollDieserEintragWirklichGeloeschtWerden}?')"
-				rendered="#{ProzessverwaltungForm.myVorlageEigenschaft.id != null}">
+				rendered="#{ProzessverwaltungForm.myVorlage.displayProperties.currentProperty.id != null}">
 				<x:updateActionListener
 					property="#{ProzessverwaltungForm.modusBearbeiten}" value="" />
 			</h:commandButton>

=== modified file 'newpages/inc_Prozessverwaltung/werkstueck_box_Eigenschaften.jsp'
--- newpages/inc_Prozessverwaltung/werkstueck_box_Eigenschaften.jsp	2011-12-20 08:07:09 +0000
+++ newpages/inc_Prozessverwaltung/werkstueck_box_Eigenschaften.jsp	2012-03-06 12:25:21 +0000
@@ -39,7 +39,7 @@
 	cellspacing="1px" cellpadding="1px" headerClass="standardTable_Header"
 	rowClasses="standardTable_Row1"
 	columnClasses="standardTable_Column,standardTable_Column,standardTable_ColumnCentered"
-	var="item" value="#{ProzessverwaltungForm.myWerkstueck.eigenschaftenList}"
+	var="item" value="#{ProzessverwaltungForm.myWerkstueck.displayProperties.sortedProperties}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='werkstueckeigenschaft'}">
 
 	<h:column>
@@ -65,7 +65,7 @@
 			title="#{msgs.bearbeiten}">
 			<h:graphicImage value="/newpages/images/buttons/edit.gif" />
 			<x:updateActionListener
-				property="#{ProzessverwaltungForm.myWerkstueckEigenschaft}"
+				property="#{ProzessverwaltungForm.myWerkstueck.displayProperties.currentProperty}"
 				value="#{item}" />
 			<x:updateActionListener
 				property="#{ProzessverwaltungForm.modusBearbeiten}"
@@ -75,7 +75,7 @@
 </x:dataTable>
 
 <%-- Neu-Schaltknopf --%>
-<h:commandLink action="#{ProzessverwaltungForm.WerkstueckEigenschaftNeu}"
+<h:commandLink action="#{ProzessverwaltungForm.myWerkstueck.displayProperties.createNewProperty}"
 	value="#{msgs.eigenschaftHinzufuegen}"
 	title="#{msgs.eigenschaftHinzufuegen}"
 	rendered="#{ProzessverwaltungForm.modusBearbeiten!='werkstueckeigenschaft'}">
@@ -114,7 +114,7 @@
 					<h:panelGroup>
 						<h:inputText id="eigenschafttitel"
 							style="width: 300px;margin-right:15px"
-							value="#{ProzessverwaltungForm.myWerkstueckEigenschaft.titel}"
+							value="#{ProzessverwaltungForm.myWerkstueck.displayProperties.currentProperty.titel}"
 							required="true" />
 						<x:message for="eigenschafttitel" style="color: red"
 							detailFormat="#{msgs.keinTitelAngegeben}" />
@@ -124,7 +124,7 @@
 					<h:panelGroup>
 						<h:inputText id="eigenschaftwert"
 							style="width: 300px;margin-right:15px"
-							value="#{ProzessverwaltungForm.myWerkstueckEigenschaft.wert}" />
+							value="#{ProzessverwaltungForm.myWerkstueck.displayProperties.currentProperty.wert}" />
 					</h:panelGroup>
 				</h:panelGrid>
 
@@ -141,9 +141,9 @@
 			</htm:td>
 			<htm:td styleClass="eingabeBoxen_row3" align="right">
 				<h:commandButton value="#{msgs.loeschen}"
-					action="#{ProzessverwaltungForm.WerkstueckEigenschaftLoeschen}"
+					action="#{ProzessverwaltungForm.myWerkstueck.displayProperties.deleteProperty}"
 					onclick="return confirm('#{msgs.sollDieserEintragWirklichGeloeschtWerden}?')"
-					rendered="#{ProzessverwaltungForm.myWerkstueckEigenschaft.id != null}">
+					rendered="#{ProzessverwaltungForm.myWerkstueck.displayProperties.currentProperty.id != null}">
 					<x:updateActionListener
 						property="#{ProzessverwaltungForm.modusBearbeiten}" value="" />
 				</h:commandButton>

=== modified file 'src/de/sub/goobi/forms/ProzessverwaltungForm.java'
--- src/de/sub/goobi/forms/ProzessverwaltungForm.java	2012-02-22 07:43:02 +0000
+++ src/de/sub/goobi/forms/ProzessverwaltungForm.java	2012-03-06 12:25:21 +0000
@@ -375,50 +375,6 @@
 	 * Eigenschaften
 	 */
 
-	public String ProzessEigenschaftLoeschen() {
-		try {
-			myProzess.getEigenschaften().remove(myProzessEigenschaft);
-			dao.save(myProzess);
-		} catch (DAOException e) {
-			Helper.setFehlerMeldung("fehlerNichtLoeschbar", e.getMessage());
-		}
-		myProzess.refreshProperties();
-		return "";
-	}
-
-	public String SchrittEigenschaftLoeschen() {
-		try {
-			mySchritt.getEigenschaften().remove(mySchrittEigenschaft);
-			dao.save(myProzess);
-		} catch (DAOException e) {
-			Helper.setFehlerMeldung("fehlerNichtLoeschbar", e.getMessage());
-		}
-		mySchritt.refreshProperties();
-		return "";
-	}
-
-	public String VorlageEigenschaftLoeschen() {
-		try {
-			myVorlage.getEigenschaften().remove(myVorlageEigenschaft);
-			dao.save(myProzess);
-		} catch (DAOException e) {
-			Helper.setFehlerMeldung("fehlerNichtLoeschbar", e.getMessage());
-		}
-		myVorlage.refreshProperties();
-		return "";
-	}
-
-	public String WerkstueckEigenschaftLoeschen() {
-		try {
-			myWerkstueck.getEigenschaften().remove(myWerkstueckEigenschaft);
-			dao.save(myProzess);
-		} catch (DAOException e) {
-			Helper.setFehlerMeldung("fehlerNichtLoeschbar", e.getMessage());
-		}
-		myWerkstueck.refreshProperties();
-		return "";
-	}
-
 	public String ProzessEigenschaftUebernehmen() {
 		myProzess.getEigenschaften().add((Prozesseigenschaft) myProzess.getDisplayProperties().getCurrentProperty().getProperty());
 		((Prozesseigenschaft) myProzess.getDisplayProperties().getCurrentProperty().getProperty()).setProzess(myProzess);