Skip to content
Snippets Groups Projects
Verified Commit 59108256 authored by Timm Fitschen's avatar Timm Fitschen
Browse files

BUG: Fix #217 and #221

parent 400e1c18
Branches
Tags
2 merge requests!58REL: prepare release 0.7.2,!56Fix #217 and #221
...@@ -399,6 +399,8 @@ public class WriteTransaction extends Transaction<WritableContainer> ...@@ -399,6 +399,8 @@ public class WriteTransaction extends Transaction<WritableContainer>
|| newEntity.hasDatatype() ^ oldEntity.hasDatatype()) { || newEntity.hasDatatype() ^ oldEntity.hasDatatype()) {
needPermissions.add(EntityPermission.UPDATE_DATA_TYPE); needPermissions.add(EntityPermission.UPDATE_DATA_TYPE);
updatetable = true; updatetable = true;
} else {
newEntity.setDatatypeOverride(oldEntity.isDatatypeOverride());
} }
// entity role // entity role
...@@ -417,7 +419,7 @@ public class WriteTransaction extends Transaction<WritableContainer> ...@@ -417,7 +419,7 @@ public class WriteTransaction extends Transaction<WritableContainer>
try { try {
newEntity.parseValue(); newEntity.parseValue();
oldEntity.parseValue(); oldEntity.parseValue();
} catch (Message m) { } catch (NullPointerException|Message m) {
// ignore, parsing is handled elsewhere // ignore, parsing is handled elsewhere
} }
if (!newEntity.getValue().equals(oldEntity.getValue())) { if (!newEntity.getValue().equals(oldEntity.getValue())) {
...@@ -436,6 +438,8 @@ public class WriteTransaction extends Transaction<WritableContainer> ...@@ -436,6 +438,8 @@ public class WriteTransaction extends Transaction<WritableContainer>
|| newEntity.hasName() ^ oldEntity.hasName()) { || newEntity.hasName() ^ oldEntity.hasName()) {
needPermissions.add(EntityPermission.UPDATE_NAME); needPermissions.add(EntityPermission.UPDATE_NAME);
updatetable = true; updatetable = true;
} else {
newEntity.setNameOverride(oldEntity.isNameOverride());
} }
// entity description // entity description
...@@ -445,6 +449,8 @@ public class WriteTransaction extends Transaction<WritableContainer> ...@@ -445,6 +449,8 @@ public class WriteTransaction extends Transaction<WritableContainer>
|| newEntity.hasDescription() ^ oldEntity.hasDescription()) { || newEntity.hasDescription() ^ oldEntity.hasDescription()) {
needPermissions.add(EntityPermission.UPDATE_DESCRIPTION); needPermissions.add(EntityPermission.UPDATE_DESCRIPTION);
updatetable = true; updatetable = true;
} else {
newEntity.setDescOverride(oldEntity.isDescOverride());
} }
// file properties // file properties
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment