Use the -m option in xfconf-query to monitor for changes #5

2022-09-18T22:06:23+02:00

rokosun commented

2022-09-18 22:06:23 +02:00

(Migrated from git.webape.site)

I also improved the overall code syntax, there's no need for the old functions anymore.

❤️ 2

rokosun commented

2022-09-18 22:11:55 +02:00

(Migrated from git.webape.site)

@tio: Test this and let me know if you find any bugs, when you're satisfied with the script then you can merge it.

👍 1

eSh (Migrated from git.webape.site) approved these changes

2022-09-19 00:00:20 +02:00

bin/periodic/fix-theming

					
				@ -229,1 +3,3 @@

				done < <(xfconf-query -c xsettings -m)

					# Check if the system theme has been changed

					if [ "$line" = 'set: /Net/ThemeName' ]; then

eSh (Migrated from git.webape.site) commented

2022-09-19 00:00:20 +02:00

A minor thing: to replace if;then;fi with switch;case statement:

case EXPRESSION in

  PATTERN_1)
    STATEMENTS
    ;;

  PATTERN_2)
    STATEMENTS
    ;;

  PATTERN_N)
    STATEMENTS
    ;;

  *)
    STATEMENTS
    ;;
esac

More details and examples are here.

This just adds some structure and also an easier to update if some more settings changes will be necessary to subsctibe to. But I leave it up to you to decide, you can skip it if you want 😉

A minor thing: to replace `if;then;fi` with `switch;case` statement: ``` case EXPRESSION in PATTERN_1) STATEMENTS ;; PATTERN_2) STATEMENTS ;; PATTERN_N) STATEMENTS ;; *) STATEMENTS ;; esac ``` More details and examples are [here](https://linuxize.com/post/bash-case-statement/). This just adds some structure and also an easier to update if some more settings changes will be necessary to subsctibe to. But I leave it up to you to decide, you can skip it if you want 😉

eSh (Migrated from git.webape.site) commented

2022-09-19 00:36:11 +02:00

Oh of course it makes sense to use a case statement here, why didn't I think of it before? lol 😄