dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #10690
[Branch ~dhis2-devs-core/dhis2/trunk] Rev 2933: Minor fix
------------------------------------------------------------
revno: 2933
committer: Lars Helge Overland <larshelge@xxxxxxxxx>
branch nick: dhis2
timestamp: Mon 2011-02-28 18:34:25 +0100
message:
Minor fix
modified:
dhis-2/dhis-api/src/main/java/org/hisp/dhis/reporttable/ReportTable.java
dhis-2/dhis-services/dhis-service-reporting/src/main/java/org/hisp/dhis/reporttable/impl/DefaultReportTableService.java
dhis-2/dhis-web/dhis-web-reporting/src/main/java/org/hisp/dhis/reporting/tablecreator/action/GetTableOptionsAction.java
dhis-2/dhis-web/dhis-web-reporting/src/main/webapp/dhis-web-reporting/addTableForm.vm
--
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/reporttable/ReportTable.java'
--- dhis-2/dhis-api/src/main/java/org/hisp/dhis/reporttable/ReportTable.java 2011-02-24 14:45:57 +0000
+++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/reporttable/ReportTable.java 2011-02-28 17:34:25 +0000
@@ -118,7 +118,7 @@
private static final String EMPTY = "";
private static final IdentifiableObject[] IRT = new IdentifiableObject[0];
private static final String[] SRT = new String[0];
- private static final String ILLEGAL_FILENAME_CHARS_REGEX = "[/\\?%*:|\"<>.]";
+ private static final String ILLEGAL_FILENAME_CHARS_REGEX = "[/\\?%*:|\"'<>.]";
// -------------------------------------------------------------------------
// Persisted properties
@@ -548,6 +548,8 @@
{
if ( string != null )
{
+ string = string.replaceAll( "<", "_lt" );
+ string = string.replaceAll( ">", "_gt" );
string = string.replaceAll( ILLEGAL_FILENAME_CHARS_REGEX, EMPTY );
string = string.length() > 255 ? string.substring( 0, 255 ) : string;
string = string.toLowerCase();
=== modified file 'dhis-2/dhis-services/dhis-service-reporting/src/main/java/org/hisp/dhis/reporttable/impl/DefaultReportTableService.java'
--- dhis-2/dhis-services/dhis-service-reporting/src/main/java/org/hisp/dhis/reporttable/impl/DefaultReportTableService.java 2011-02-28 15:46:58 +0000
+++ dhis-2/dhis-services/dhis-service-reporting/src/main/java/org/hisp/dhis/reporttable/impl/DefaultReportTableService.java 2011-02-28 17:34:25 +0000
@@ -383,10 +383,10 @@
{
for ( DataElementCategoryOption categoryOption : reportTable.getCategoryCombo().getCategoryOptions() ) // TOTO skip if only one category?
{
- grid.addHeader( new GridHeader( categoryOption.getShortName(), columnEncode( categoryOption.getShortName() ), String.class.getName(), false, false ) );
+ grid.addHeader( new GridHeader( categoryOption.getShortName(), columnEncode( categoryOption.getShortName() ), Double.class.getName(), false, false ) );
}
- grid.addHeader( new GridHeader( TOTAL_COLUMN_PRETTY_NAME, TOTAL_COLUMN_NAME, String.class.getName(), false, false ) );
+ grid.addHeader( new GridHeader( TOTAL_COLUMN_PRETTY_NAME, TOTAL_COLUMN_NAME, Double.class.getName(), false, false ) );
}
// ---------------------------------------------------------------------
=== modified file 'dhis-2/dhis-web/dhis-web-reporting/src/main/java/org/hisp/dhis/reporting/tablecreator/action/GetTableOptionsAction.java'
--- dhis-2/dhis-web/dhis-web-reporting/src/main/java/org/hisp/dhis/reporting/tablecreator/action/GetTableOptionsAction.java 2011-02-18 21:50:37 +0000
+++ dhis-2/dhis-web/dhis-web-reporting/src/main/java/org/hisp/dhis/reporting/tablecreator/action/GetTableOptionsAction.java 2011-02-28 17:34:25 +0000
@@ -406,7 +406,7 @@
periods.removeAll( reportTable.getPeriods() );
organisationUnits.removeAll( reportTable.getUnits() );
-
+
selectedDataElements = reportTable.getDataElements();
selectedIndicators = reportTable.getIndicators();
=== modified file 'dhis-2/dhis-web/dhis-web-reporting/src/main/webapp/dhis-web-reporting/addTableForm.vm'
--- dhis-2/dhis-web/dhis-web-reporting/src/main/webapp/dhis-web-reporting/addTableForm.vm 2011-02-26 19:24:20 +0000
+++ dhis-2/dhis-web/dhis-web-reporting/src/main/webapp/dhis-web-reporting/addTableForm.vm 2011-02-28 17:34:25 +0000
@@ -80,7 +80,7 @@
<select id="categoryComboId" name="categoryComboId" style="min-width:640px" onchange="getCategoryComboDataElements()">
<option value="-1">[ $i18n.getString( "select_category_combo" ) ]</option>
#foreach ( $categoryCombo in $categoryCombos )
- <option value="$categoryCombo.id">$encoder.htmlEncode( $categoryCombo.name )</option>
+ <option value="$categoryCombo.id"#if( $reportTable && $reportTable.categoryCombo.id == $categoryCombo.id ) selected#end>$encoder.htmlEncode( $categoryCombo.name )</option>
#end
</select>
</td>
@@ -456,7 +456,6 @@
#if ( $dimension )
<script type="text/javascript">
- document.getElementById( "doIndicators" ).checked = true;
document.getElementById( "doIndicators" ).disabled = true;
document.getElementById( "doOrganisationUnits" ).disabled = true;
document.getElementById( "doPeriods" ).disabled = true;