From 8f23e7e863e5ca5c5eae4b97f3928ea0859d7f08 Mon Sep 17 00:00:00 2001 From: Vova Perebykivskyi Date: Thu, 27 Oct 2016 14:36:50 +0300 Subject: [PATCH 1/8] Add i18n for some strings. Code refactoring. --- .../resourceload/ResourceLoadDisplayData.java | 78 ++++++++----------- .../web/resourceload/ResourceLoadModel.java | 2 - .../src/main/resources/i18n/keys.pot | 12 +++ .../src/main/webapp/common/pageNotFound.zul | 6 +- .../subcontract/SubcontractServiceTest.java | 1 + 5 files changed, 49 insertions(+), 50 deletions(-) diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadDisplayData.java b/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadDisplayData.java index 81b6b2c13..5c53bba92 100644 --- a/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadDisplayData.java +++ b/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadDisplayData.java @@ -15,6 +15,34 @@ import org.zkoss.ganttz.util.Interval; public class ResourceLoadDisplayData { + private final List timeLines; + + private final Interval viewInterval; + + private final Paginator paginator; + + private final Callable> resourcesConsidered; + + private final Callable> assignmentsConsidered; + + public ResourceLoadDisplayData( + List timeLines, + Paginator paginator, + Callable> resourcesConsidered, + Callable> assignmentsConsidered) { + + Validate.notNull(timeLines); + Validate.notNull(paginator); + Validate.notNull(resourcesConsidered); + Validate.notNull(assignmentsConsidered); + + this.timeLines = timeLines; + this.viewInterval = getViewIntervalFrom(timeLines); + this.paginator = paginator; + this.resourcesConsidered = cached(resourcesConsidered); + this.assignmentsConsidered = cached(assignmentsConsidered); + } + private static Callable cached(Callable callable) { return new CachedCallable<>(callable); } @@ -28,6 +56,7 @@ public class ResourceLoadDisplayData { } private static class CachedCallable implements Callable { + private final Callable callable; private T result; @@ -42,47 +71,14 @@ public class ResourceLoadDisplayData { if (result != null) { return result; } - return result = callable.call(); + + result = callable.call(); + + return result; } } - private final List timeLines; - - private final Interval viewInterval; - - private final Paginator paginator; - - private final Callable> resourcesConsidered; - - private final Callable> assignmentsConsidered; - - private final LocalDate filterStart; - - private final LocalDate filterEnd; - - public ResourceLoadDisplayData( - List timeLines, - LocalDate filterStart, - LocalDate filterEnd, - Paginator paginator, - Callable> resourcesConsidered, - Callable> assignmentsConsidered) { - - Validate.notNull(timeLines); - Validate.notNull(paginator); - Validate.notNull(resourcesConsidered); - Validate.notNull(assignmentsConsidered); - - this.timeLines = timeLines; - this.filterStart = filterStart; - this.filterEnd = filterEnd; - this.viewInterval = getViewIntervalFrom(timeLines); - this.paginator = paginator; - this.resourcesConsidered = cached(resourcesConsidered); - this.assignmentsConsidered = cached(assignmentsConsidered); - } - private static Interval getViewIntervalFrom(List timeLines) { return LoadTimeLine.getIntervalFrom(timeLines); } @@ -115,12 +111,4 @@ public class ResourceLoadDisplayData { return resolve(assignmentsConsidered); } - public LocalDate getFilterStart() { - return filterStart; - } - - public LocalDate getFilterEnd() { - return filterEnd; - } - } diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadModel.java b/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadModel.java index 7d554c6cb..1bc752904 100644 --- a/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadModel.java +++ b/libreplan-webapp/src/main/java/org/libreplan/web/resourceload/ResourceLoadModel.java @@ -133,8 +133,6 @@ public class ResourceLoadModel implements IResourceLoadModel { return new ResourceLoadDisplayData( loadTimeLines, - parameters.getInitDateFilter(), - parameters.getEndDateFilter(), allocationsFinder.getPaginator(), allocationsFinder.lazilyGetResourcesIncluded(), allocationsFinder.lazilyGetAssignmentsShown()); diff --git a/libreplan-webapp/src/main/resources/i18n/keys.pot b/libreplan-webapp/src/main/resources/i18n/keys.pot index 8364de50a..103ac4509 100644 --- a/libreplan-webapp/src/main/resources/i18n/keys.pot +++ b/libreplan-webapp/src/main/resources/i18n/keys.pot @@ -9432,4 +9432,16 @@ msgstr "" #: libreplan-webapp/src/main/webapp/orders/_listOrderElementFiles.zul:32 msgid "No files" +msgstr "" + +#: libreplan-webapp/src/main/webapp/common/pageNotFound.zul:57 +msgid "Help/Info pages are not generated" +msgstr "" + +#: libreplan-webapp/src/main/webapp/common/pageNotFound.zul:59 +msgid "If you want to see help/info page then" +msgstr "" + +#: libreplan-webapp/src/main/webapp/common/pageNotFound.zul:61 +msgid "Open file HACKING.rst in project directory and read Part LibrePlan documentation generation, how to generate help/info pages" msgstr "" \ No newline at end of file diff --git a/libreplan-webapp/src/main/webapp/common/pageNotFound.zul b/libreplan-webapp/src/main/webapp/common/pageNotFound.zul index 6dd48a217..53ddabfdc 100644 --- a/libreplan-webapp/src/main/webapp/common/pageNotFound.zul +++ b/libreplan-webapp/src/main/webapp/common/pageNotFound.zul @@ -54,11 +54,11 @@ -