dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #43637
[Branch ~dhis2-devs-core/dhis2/trunk] Rev 22124: minor fixes, adds disply name (if available) when reporting back reference/object errors
------------------------------------------------------------
revno: 22124
committer: Morten Olav Hansen <morten@xxxxxxxxx>
branch nick: dhis2
timestamp: Thu 2016-03-03 12:02:44 +0700
message:
minor fixes, adds disply name (if available) when reporting back reference/object errors
modified:
dhis-2/dhis-api/src/main/java/org/hisp/dhis/preheat/PreheatIdentifier.java
dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/preheat/DefaultPreheatService.java
dhis-2/dhis-services/dhis-service-dxf2/src/main/java/org/hisp/dhis/dxf2/metadata2/objectbundle/DefaultObjectBundleService.java
--
lp:dhis2
https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk
Your team DHIS 2 developers is subscribed to branch lp:dhis2.
To unsubscribe from this branch go to https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk/+edit-subscription
=== modified file 'dhis-2/dhis-api/src/main/java/org/hisp/dhis/preheat/PreheatIdentifier.java'
--- dhis-2/dhis-api/src/main/java/org/hisp/dhis/preheat/PreheatIdentifier.java 2016-03-01 07:51:42 +0000
+++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/preheat/PreheatIdentifier.java 2016-03-03 05:02:44 +0000
@@ -30,6 +30,7 @@
import com.google.common.collect.Lists;
import org.hisp.dhis.common.IdentifiableObject;
+import org.springframework.util.StringUtils;
import java.util.ArrayList;
import java.util.List;
@@ -88,4 +89,17 @@
return new ArrayList<>();
}
+
+ public <T extends IdentifiableObject> String getIdentifiersWithName( T object )
+ {
+ List<String> identifiers = getIdentifiers( object );
+ String name = StringUtils.isEmpty( object.getDisplayName() ) ? null : object.getDisplayName();
+
+ if ( name == null )
+ {
+ return identifiers.toString();
+ }
+
+ return name + " " + identifiers.toString();
+ }
}
=== modified file 'dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/preheat/DefaultPreheatService.java'
--- dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/preheat/DefaultPreheatService.java 2016-03-03 03:10:04 +0000
+++ dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/preheat/DefaultPreheatService.java 2016-03-03 05:02:44 +0000
@@ -472,7 +472,7 @@
if ( ref == null && refObject != null && !Preheat.isDefault( refObject ) )
{
preheatErrorReports.add( new PreheatErrorReport( identifier, object.getClass(), ErrorCode.E5001,
- identifier.getIdentifiers( refObject ), identifier.getIdentifiers( object ), p.getName() ) );
+ identifier.getIdentifiersWithName( refObject ), identifier.getIdentifiersWithName( object ), p.getName() ) );
}
}
else
@@ -489,7 +489,7 @@
if ( ref == null && refObject != null )
{
preheatErrorReports.add( new PreheatErrorReport( identifier, object.getClass(), ErrorCode.E5001,
- identifier.getIdentifiers( refObject ), identifier.getIdentifiers( object ), p.getCollectionName() ) );
+ identifier.getIdentifiersWithName( refObject ), identifier.getIdentifiersWithName( object ), p.getCollectionName() ) );
}
else
{
=== modified file 'dhis-2/dhis-services/dhis-service-dxf2/src/main/java/org/hisp/dhis/dxf2/metadata2/objectbundle/DefaultObjectBundleService.java'
--- dhis-2/dhis-services/dhis-service-dxf2/src/main/java/org/hisp/dhis/dxf2/metadata2/objectbundle/DefaultObjectBundleService.java 2016-03-03 03:10:04 +0000
+++ dhis-2/dhis-services/dhis-service-dxf2/src/main/java/org/hisp/dhis/dxf2/metadata2/objectbundle/DefaultObjectBundleService.java 2016-03-03 05:02:44 +0000
@@ -185,7 +185,7 @@
if ( object == null )
{
objectBundleValidation.addErrorReport( klass, ErrorCode.E5000, bundle.getPreheatIdentifier(),
- bundle.getPreheatIdentifier().getIdentifiers( identifiableObject ) );
+ bundle.getPreheatIdentifier().getIdentifiersWithName( identifiableObject ) );
iterator.remove();
}
}