Merge pull request #587 from hisptz/bugfix/specific-targets-library

fix(Scorecard PDF Download): Download title
hisptz · Feb 25, 2022 · bd34094 · bd34094
2 parents 909fc6a + a6fe4d9
commit bd34094
Showing 1 changed file with 51 additions and 41 deletions.
diff --git a/src/modules/Main/Components/ScorecardView/Components/SpecificTargetsLibrary/index.js b/src/modules/Main/Components/ScorecardView/Components/SpecificTargetsLibrary/index.js
@@ -23,48 +23,58 @@ export function SpecificTargetsLibrary() {
         return data;
     }, [dataSources])
     return (
-        <div className="column gap-16">
-            <div>
-                <h3>{i18n.t("Organisation Units Specific targets")}</h3>
-                <div className="row gap-16">
-                    {
-                        specificTargetsDataSourcesByType?.orgUnit?.map(dataSource => <>
-                            <OrgUnitSpecificTargetView
-                                key={`${dataSource.id}-orgUnit-specific-target`}
-                                specificTarget={head(dataSource.specificTargets)} dataSourceLabel={dataSource.label}/>
-                            <div className="page-break"/>
-                        </>)
-                    }
-                </div>
-                <div className="page-break"/>
+        <>
+            <div className="column gap-16">
+                {
+                    !isEmpty(specificTargetsDataSourcesByType?.orgUnit) && <div>
+                        <h3>{i18n.t("Organisation Units Specific targets")}</h3>
+                        <div className="row gap-16">
+                            {
+                                specificTargetsDataSourcesByType?.orgUnit?.map(dataSource => <>
+                                    <OrgUnitSpecificTargetView
+                                        key={`${dataSource.id}-orgUnit-specific-target`}
+                                        specificTarget={head(dataSource.specificTargets)}
+                                        dataSourceLabel={dataSource.label}/>
+                                    <div className="page-break"/>
+                                </>)
+                            }
+                        </div>
+                        <div className="page-break"/>
+                    </div>
+                }
+                {
+                    !isEmpty(specificTargetsDataSourcesByType?.period) && <div>
+                        <h3>{i18n.t("Period Specific targets")}</h3>
+                        <div className="row gap-16">
+                            {
+                                specificTargetsDataSourcesByType?.period?.map(dataSource => <>
+                                    <PeriodSpecificTargetView
+                                        key={`${dataSource.id}-orgUnit-specific-target`}
+                                        specificTarget={head(dataSource.specificTargets)}
+                                        dataSourceLabel={dataSource.label}/>
+                                    <div className="page-break"/>
+                                </>)
+                            }
+                        </div>
+                        <div className="page-break"/>
+                    </div>
+                }
+                {
+                    !isEmpty(specificTargetsDataSourcesByType?.orgUnitLevell) && <div>
+                        <h3>{i18n.t("Organisation unit level targets")}</h3>
+                        <div className="column gap-16">
+                            {
+                                specificTargetsDataSourcesByType?.orgUnitLevel?.map(dataSource =>
+                                    <OrgUnitLevelSpecificTargetView
+                                        key={`${dataSource.id}-orgUnit-specific-target`}
+                                        legends={dataSource.legends} dataSourceLabel={dataSource.label}/>)
+                            }
+                        </div>
+                        <div className="page-break"/>
+                    </div>
+                }
             </div>
-            <div>
-                <h3>{i18n.t("Period Specific targets")}</h3>
-                <div className="row gap-16">
-                    {
-                        specificTargetsDataSourcesByType?.period?.map(dataSource => <>
-                            <PeriodSpecificTargetView
-                                key={`${dataSource.id}-orgUnit-specific-target`}
-                                specificTarget={head(dataSource.specificTargets)} dataSourceLabel={dataSource.label}/>
-                            <div className="page-break"/>
-                        </>)
-                    }
-                </div>
-                <div className="page-break"/>
-            </div>
-            <div>
-                <h3>{i18n.t("Organisation unit level targets")}</h3>
-                <div className="column gap-16">
-                    {
-                        specificTargetsDataSourcesByType?.orgUnitLevel?.map(dataSource =>
-                            <OrgUnitLevelSpecificTargetView
-                                key={`${dataSource.id}-orgUnit-specific-target`}
-                                legends={dataSource.legends} dataSourceLabel={dataSource.label}/>)
-                    }
-                </div>
-                <div className="page-break"/>
-            </div>
-        </div>
+        </>
     )
 }