From 9eedbede7466be71de6fa39968bb15fef99eb19b Mon Sep 17 00:00:00 2001
From: Maximilian Zettler <maximilian.zettler@mawoh.de>
Date: Fri, 24 Aug 2018 14:20:43 +0200
Subject: [PATCH 1/2] fix polkit state include

---
 users/init.sls | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/users/init.sls b/users/init.sls
index 8bcfbab..5a9a8d6 100644
--- a/users/init.sls
+++ b/users/init.sls
@@ -3,6 +3,7 @@
 {% set used_sudo = [] %}
 {% set used_googleauth = [] %}
 {% set used_user_files = [] %}
+{% set used_polkit = False %}
 
 {%- for name, user in pillar.get('users', {}).items()
         if user.absent is not defined or not user.absent %}
@@ -18,9 +19,12 @@
 {%- if salt['pillar.get']('users:' ~ name ~ ':user_files:enabled', False) %}
 {%- do used_user_files.append(1) %}
 {%- endif %}
+{%- if user.get('polkitadmin', False) == True %}
+{%- set used_polkit = True %}
+{%- endif %}
 {%- endfor %}
 
-{%- if used_sudo or used_googleauth or used_user_files %}
+{%- if used_sudo or used_googleauth or used_user_files or used_polkit %}
 include:
 {%- if used_sudo %}
   - users.sudo
@@ -31,8 +35,10 @@ include:
 {%- if used_user_files %}
   - users.user_files
 {%- endif %}
-{%- endif %}
+{%- if used_polkit %}
   - users.polkit
+{%- endif %}
+{%- endif %}
 
 {% for name, user in pillar.get('users', {}).items()
         if user.absent is not defined or not user.absent %}

From 0e93ebccdb857c5860c89cfaa3d0f0c45fc77b11 Mon Sep 17 00:00:00 2001
From: Maximilian Zettler <maximilian.zettler@mawoh.de>
Date: Fri, 24 Aug 2018 14:25:35 +0200
Subject: [PATCH 2/2] use do to update dict

---
 users/polkit.sls | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/users/polkit.sls b/users/polkit.sls
index df959bc..fd8a5ed 100644
--- a/users/polkit.sls
+++ b/users/polkit.sls
@@ -5,8 +5,7 @@
 {% for name, user in pillar.get('users', {}).items() %}
   {% if user.absent is not defined or not user.absent %}
     {% if 'polkitadmin' in user and user['polkitadmin'] %}
-      {% if polkitusers.update({'value': polkitusers.value + 'unix-user:' + name + ';'}) %}
-      {% endif %}
+      {% do polkitusers.update({'value': polkitusers.value + 'unix-user:' + name + ';'}) %}
     {% endif %}
   {% endif %}
 {% endfor %}