summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLionel Elie Mamane <lionel@mamane.lu>2013-02-19 12:27:18 +0100
committerLionel Elie Mamane <lionel@mamane.lu>2013-02-19 12:35:03 +0100
commit9f015df3558ddda6729fd0078cd3922feaa65aac (patch)
tree88367ccdc9a49934de37e5a29d3afe5c828b89c8
parentae1fbbfb690cb35eae7d56ae190d637d834697aa (diff)
add comment
Change-Id: Id14dd289f8803378838725fbe08e327c9b1d233d
-rw-r--r--reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java12
1 files changed, 12 insertions, 0 deletions
diff --git a/reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java b/reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java
index 83c572a2591b..7a7465f99345 100644
--- a/reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java
+++ b/reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java
@@ -109,6 +109,18 @@ public abstract class AbstractReportElementLayoutController
(AbstractReportElementLayoutController) clone();
alc.state = AbstractReportElementLayoutController.FINISHED;
return alc;
+ // That's how this method is implemented in classes of pentaho itself;
+ // I'm not sure why we do something different, but I haven't been able
+ // to pinpoint a bug attributable to the above implementation.
+ // final LayoutController parent = getParent();
+ // if (parent == null)
+ // {
+ // // skip to the next step ..
+ // throw new IllegalStateException("There is no parent to join with. " +
+ // "This should not happen in a sane environment!");
+ // }
+
+ // return parent.join(getFlowController());
}
protected abstract LayoutController delegateContentGeneration(final ReportTarget target)