From 4443518a6349e37e039266de69666a58d79ba70a Mon Sep 17 00:00:00 2001
From: Imran Iqbal <iqbalmy@hotmail.com>
Date: Mon, 25 Feb 2019 15:27:24 +0000
Subject: [PATCH] refactor(include+require): use variable for duplicate values

---
 template/config/clean.sls    | 5 +++--
 template/config/file.sls     | 5 +++--
 template/pkg/clean.sls       | 5 +++--
 template/service/running.sls | 5 +++--
 4 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/template/config/clean.sls b/template/config/clean.sls
index ad0802dc..f3ee871c 100644
--- a/template/config/clean.sls
+++ b/template/config/clean.sls
@@ -3,14 +3,15 @@
 
 {#- Get the `topdir` from `tpldir` #}
 {%- set topdir = tpldir.split('/')[0] %}
+{%- set sls_service_clean = topdir ~ '.service.clean' %}
 {%- from topdir ~ "/map.jinja" import template with context %}
 {%- from topdir ~ "/macros.jinja" import files_switch with context %}
 
 include:
-  - {{ topdir }}.service.clean
+  - {{ sls_service_clean }}
 
 template-config-absent:
   file.absent:
     - name: {{ template.config }}
     - require:
-      - sls: {{ topdir }}.service.clean
+      - sls: {{ sls_service_clean }}
diff --git a/template/config/file.sls b/template/config/file.sls
index e9a4a290..aca3a7fa 100644
--- a/template/config/file.sls
+++ b/template/config/file.sls
@@ -3,11 +3,12 @@
 
 {#- Get the `topdir` from `tpldir` #}
 {%- set topdir = tpldir.split('/')[0] %}
+{%- set sls_pkg_install = topdir ~ '.pkg.install' %}
 {%- from topdir ~ "/map.jinja" import template with context %}
 {%- from topdir ~ "/macros.jinja" import files_switch with context %}
 
 include:
-  - {{ topdir }}.pkg.install
+  - {{ sls_pkg_install }}
 
 template-config:
   file.managed:
@@ -23,4 +24,4 @@ template-config:
     - group: root
     - template: jinja
     - require:
-      - sls: {{ topdir }}.pkg.install
+      - sls: {{ sls_pkg_install }}
diff --git a/template/pkg/clean.sls b/template/pkg/clean.sls
index da522ceb..74a088e0 100644
--- a/template/pkg/clean.sls
+++ b/template/pkg/clean.sls
@@ -3,13 +3,14 @@
 
 {#- Get the `topdir` from `tpldir` #}
 {%- set topdir = tpldir.split('/')[0] %}
+{%- set sls_config_clean = topdir ~ '.config.clean' %}
 {%- from topdir ~ "/map.jinja" import template with context %}
 
 include:
-  - {{ topdir }}.config.clean
+  - {{ sls_config_clean }}
 
 template-pkg-removed:
   pkg.removed:
     - name: {{ template.pkg }}
     - require:
-      - sls: {{ topdir }}.config.clean
+      - sls: {{ sls_config_clean }}
diff --git a/template/service/running.sls b/template/service/running.sls
index 963dae8a..5651744b 100644
--- a/template/service/running.sls
+++ b/template/service/running.sls
@@ -3,14 +3,15 @@
 
 {#- Get the `topdir` from `tpldir` #}
 {%- set topdir = tpldir.split('/')[0] %}
+{%- set sls_config_file = topdir ~ '.config.file' %}
 {%- from topdir ~ "/map.jinja" import template with context %}
 
 include:
-  - {{ topdir }}.config.file
+  - {{ sls_config_file }}
 
 template-service:
   service.running:
     - name: {{ template.service.name }}
     - enable: True
     - require:
-      - sls: {{ topdir }}.config.file
+      - sls: {{ sls_config_file }}