Read before writing smc

This reverts commit 4ced07c359.

LICENSE.md

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2024  Santiago Lo Coco
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

cbattery

@@ -9,6 +9,7 @@ PATH="/usr/bin:/usr/local/bin:/usr/sbin:/opt/homebrew/bin:/opt/homebrew/sbin:/op
 usage() {
     cat << EOF
 usage: ${0##*/} [command]
+
     charging [on|off]		- Toggle charging
     adapter [on|off]		- Toggle adapter connection
     status			- Get status information
@@ -26,24 +27,31 @@ read_cache() {
     cat "$TMP_FILE" 2> /dev/null
 }
 
+log_message() {
+    timestamp=$(date "+%b %d %H:%M:%S")
+    echo "$timestamp - $1"
+}
+
 enable_discharging() {
-    echo "Enabling battery discharging"
+    log_message "Enabling battery discharging"
     sudo smc -k CH0I -w 01
 }
 
 disable_discharging() {
-    echo "Disabling battery discharging"
+    log_message "Disabling battery discharging"
     sudo smc -k CH0I -w 00
 }
 
 enable_charging() {
-    echo "Enabling battery charging"
+    [[ "$(get_smc_charging_hex)" == "00" ]] && return
+    log_message "Enabling battery charging"
     sudo smc -k CH0B -w 00
     sudo smc -k CH0C -w 00
 }
 
 disable_charging() {
-    echo "Disabling battery charging"
+    [[ "$(get_smc_charging_hex)" != "00" ]] && return
+    log_message "Disabling battery charging"
     sudo smc -k CH0B -w 02
     sudo smc -k CH0C -w 02
 }
@@ -78,7 +86,7 @@ install_visudo_file() {
 
 install_launch_daemon() {
     launch_daemon=/Library/LaunchDaemons/cbattery.plist
-    script_dir="$(dirname "$(readlink -f "$0")")"
+    script_dir="$(dirname "$(realpath "$0")")"
     share_plist="$SHARE_DIR/cbattery.plist"
     if ! grep -q "$script_dir" "$share_plist"; then
 	sed "s:/usr/local/bin:$script_dir:g" "$share_plist" | sudo tee "$share_plist" > /dev/null
@@ -117,7 +125,7 @@ runScript() {
 	    [[ -f "$launch_daemon" ]] && sudo launchctl unload -w "$launch_daemon"
             ;;
         "charging")
-            echo "Setting $action to $setting"
+            log_message "Setting $action to $setting"
             case "$setting" in
                 "on") enable_charging && write_cache "enabled" ;;
                 "off") disable_charging && write_cache "disabled" ;;
@@ -125,7 +133,7 @@ runScript() {
             esac
             ;;
         "adapter")
-            echo "Setting $action to $setting"
+            log_message "Setting $action to $setting"
             case "$setting" in
                 "on") enable_discharging ;;
                 "off") disable_discharging ;;
@@ -135,7 +143,7 @@ runScript() {
         "status")
             case "$setting" in
                 "charging") exit "$(get_smc_charging_hex)" ;;
-                "cache") echo "$(get_cached_status)" ;;
+                "cache") get_cached_status ;;
                 *) echo "Battery at $(get_battery_percentage)%, smc charging $(get_smc_charging_status)" ;;
             esac
             ;;