📝 Adds screenshot to README

The latest version is now visible at the top.

README.md

@@ -1,3 +1,30 @@
 # phpmon
 
-phpmon is a macOS utility that runs on your Mac and displays the active PHP version in your status bar. Handy if you're running multiple versions of PHP with Homebrew and wish to see which version is currently linked.
+<img src="./docs/phpmon.png" width="306px" alt="phpmon screenshot"/>
+
+phpmon is a macOS utility that runs on your Mac and displays the active PHP version in your status bar. Handy if you're running multiple versions of PHP with Homebrew and wish to see which version is currently linked & active with Laravel Valet.
+
+## How it works
+
+### Version detection
+
+This utility runs `php -r 'print phpversion();'` in the background periodically and extracts the version number.
+
+### Switching PHP versions
+
+This utility will detect which PHP versions you have installed via Homebrew, and then allows you to switch between them.
+
+This means:
+
+- You have at least the latest version of PHP installed (`php@7.3`)
+- You have installed Laravel Valet (`which valet` returns `/usr/local/bin/valet`)
+- You ran `valet trust`, which means Valet commands can be run without using sudo
+
+The utility runs the following commands:
+
+- Unlink all detected PHP versions
+- Switch to PHP 7.3 (this is done in order to ensure that Valet works, even when attempting to use PHP 5.6)
+- Tell Valet to switch to a specific PHP version
+- Link the desired version of PHP
+
+If you want to know more about how this works, you can find the file Services.swift under the Helpers directory to learn more about how switching PHP versions works. In the end, this just executes some shell commands.

phpmon.xcodeproj/project.pbxproj

@@ -13,10 +13,10 @@
 		C41C1B3E22B0098000E7CF16 /* Main.storyboard in Resources */ = {isa = PBXBuildFile; fileRef = C41C1B3C22B0098000E7CF16 /* Main.storyboard */; };
 		C41C1B4722B009A400E7CF16 /* Shell.swift in Sources */ = {isa = PBXBuildFile; fileRef = C41C1B4622B009A400E7CF16 /* Shell.swift */; };
 		C41C1B4922B00A9800E7CF16 /* ImageGenerator.swift in Sources */ = {isa = PBXBuildFile; fileRef = C41C1B4822B00A9800E7CF16 /* ImageGenerator.swift */; };
-		C41C1B4B22B019FF00E7CF16 /* PHPVersion.swift in Sources */ = {isa = PBXBuildFile; fileRef = C41C1B4A22B019FF00E7CF16 /* PHPVersion.swift */; };
+		C41C1B4B22B019FF00E7CF16 /* PhpVersion.swift in Sources */ = {isa = PBXBuildFile; fileRef = C41C1B4A22B019FF00E7CF16 /* PhpVersion.swift */; };
 		C41C1B4D22B0215A00E7CF16 /* Services.swift in Sources */ = {isa = PBXBuildFile; fileRef = C41C1B4C22B0215A00E7CF16 /* Services.swift */; };
 		C476FF9822B0DD830098105B /* Alert.swift in Sources */ = {isa = PBXBuildFile; fileRef = C476FF9722B0DD830098105B /* Alert.swift */; };
-		C4D8016622B1584700C6DA1B /* Environment.swift in Sources */ = {isa = PBXBuildFile; fileRef = C4D8016522B1584700C6DA1B /* Environment.swift */; };
+		C4D8016622B1584700C6DA1B /* BootChecks.swift in Sources */ = {isa = PBXBuildFile; fileRef = C4D8016522B1584700C6DA1B /* BootChecks.swift */; };
 /* End PBXBuildFile section */
 
 /* Begin PBXFileReference section */
@@ -29,10 +29,10 @@
 		C41C1B4022B0098000E7CF16 /* phpmon.entitlements */ = {isa = PBXFileReference; lastKnownFileType = text.plist.entitlements; path = phpmon.entitlements; sourceTree = "<group>"; };
 		C41C1B4622B009A400E7CF16 /* Shell.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Shell.swift; sourceTree = "<group>"; };
 		C41C1B4822B00A9800E7CF16 /* ImageGenerator.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ImageGenerator.swift; sourceTree = "<group>"; };
-		C41C1B4A22B019FF00E7CF16 /* PHPVersion.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PHPVersion.swift; sourceTree = "<group>"; };
+		C41C1B4A22B019FF00E7CF16 /* PhpVersion.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PhpVersion.swift; sourceTree = "<group>"; };
 		C41C1B4C22B0215A00E7CF16 /* Services.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Services.swift; sourceTree = "<group>"; };
 		C476FF9722B0DD830098105B /* Alert.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Alert.swift; sourceTree = "<group>"; };
-		C4D8016522B1584700C6DA1B /* Environment.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Environment.swift; sourceTree = "<group>"; };
+		C4D8016522B1584700C6DA1B /* BootChecks.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = BootChecks.swift; sourceTree = "<group>"; };
 /* End PBXFileReference section */
 
 /* Begin PBXFrameworksBuildPhase section */
@@ -65,13 +65,15 @@
 		C41C1B3522B0097F00E7CF16 /* phpmon */ = {
 			isa = PBXGroup;
 			children = (
+				C41E181722CB61EB0072CF09 /* Classes */,
+				C41E181622CB61890072CF09 /* Startup */,
+				C41E181522CB614C0072CF09 /* Terminal */,
+				C41C1B4E22B024F100E7CF16 /* Helpers */,
+				C41E181822CB62200072CF09 /* View Controllers */,
 				C41C1B3622B0097F00E7CF16 /* AppDelegate.swift */,
-				C41C1B3822B0097F00E7CF16 /* ViewController.swift */,
-				C41C1B3A22B0098000E7CF16 /* Assets.xcassets */,
-				C41C1B3C22B0098000E7CF16 /* Main.storyboard */,
 				C41C1B3F22B0098000E7CF16 /* Info.plist */,
 				C41C1B4022B0098000E7CF16 /* phpmon.entitlements */,
-				C41C1B4E22B024F100E7CF16 /* Helpers */,
+				C41C1B3A22B0098000E7CF16 /* Assets.xcassets */,
 			);
 			path = phpmon;
 			sourceTree = "<group>";
@@ -79,16 +81,46 @@
 		C41C1B4E22B024F100E7CF16 /* Helpers */ = {
 			isa = PBXGroup;
 			children = (
-				C41C1B4622B009A400E7CF16 /* Shell.swift */,
 				C41C1B4822B00A9800E7CF16 /* ImageGenerator.swift */,
-				C41C1B4A22B019FF00E7CF16 /* PHPVersion.swift */,
-				C41C1B4C22B0215A00E7CF16 /* Services.swift */,
 				C476FF9722B0DD830098105B /* Alert.swift */,
-				C4D8016522B1584700C6DA1B /* Environment.swift */,
 			);
 			path = Helpers;
 			sourceTree = "<group>";
 		};
+		C41E181522CB614C0072CF09 /* Terminal */ = {
+			isa = PBXGroup;
+			children = (
+				C41C1B4622B009A400E7CF16 /* Shell.swift */,
+				C41C1B4C22B0215A00E7CF16 /* Services.swift */,
+			);
+			path = Terminal;
+			sourceTree = "<group>";
+		};
+		C41E181622CB61890072CF09 /* Startup */ = {
+			isa = PBXGroup;
+			children = (
+				C4D8016522B1584700C6DA1B /* BootChecks.swift */,
+			);
+			path = Startup;
+			sourceTree = "<group>";
+		};
+		C41E181722CB61EB0072CF09 /* Classes */ = {
+			isa = PBXGroup;
+			children = (
+				C41C1B4A22B019FF00E7CF16 /* PhpVersion.swift */,
+			);
+			path = Classes;
+			sourceTree = "<group>";
+		};
+		C41E181822CB62200072CF09 /* View Controllers */ = {
+			isa = PBXGroup;
+			children = (
+				C41C1B3C22B0098000E7CF16 /* Main.storyboard */,
+				C41C1B3822B0097F00E7CF16 /* ViewController.swift */,
+			);
+			path = "View Controllers";
+			sourceTree = "<group>";
+		};
 /* End PBXGroup section */
 
 /* Begin PBXNativeTarget section */
@@ -159,13 +191,13 @@
 			isa = PBXSourcesBuildPhase;
 			buildActionMask = 2147483647;
 			files = (
-				C4D8016622B1584700C6DA1B /* Environment.swift in Sources */,
+				C4D8016622B1584700C6DA1B /* BootChecks.swift in Sources */,
 				C41C1B4722B009A400E7CF16 /* Shell.swift in Sources */,
 				C41C1B4D22B0215A00E7CF16 /* Services.swift in Sources */,
 				C41C1B4922B00A9800E7CF16 /* ImageGenerator.swift in Sources */,
 				C41C1B3922B0097F00E7CF16 /* ViewController.swift in Sources */,
 				C41C1B3722B0097F00E7CF16 /* AppDelegate.swift in Sources */,
-				C41C1B4B22B019FF00E7CF16 /* PHPVersion.swift in Sources */,
+				C41C1B4B22B019FF00E7CF16 /* PhpVersion.swift in Sources */,
 				C476FF9822B0DD830098105B /* Alert.swift in Sources */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;

phpmon/AppDelegate.swift

@@ -11,21 +11,22 @@ import Cocoa
 @NSApplicationMain
 class AppDelegate: NSObject, NSApplicationDelegate {
     
-    let statusItem = NSStatusBar.system.statusItem(
-        withLength: 32
-    )
+    // MARK: - Variables
     
+    let statusItem = NSStatusBar.system.statusItem(withLength: 32)
     var timer: Timer?
-    var version: PHPVersion? = nil
+    var version: PhpVersion? = nil
     var availablePhpVersions : [String] = []
     var busy: Bool = false
     
+    // MARK: - Lifecycle
+
     func applicationDidFinishLaunching(_ aNotification: Notification) {
         // Start with the icon
         self.setStatusBar(image: NSImage(named: NSImage.Name("StatusBarIcon"))!)
         // Perform environment boot checks
          DispatchQueue.global(qos: .userInitiated).async { [unowned self] in
-            Environment.performBootChecks()
+            BootChecks.perform()
             self.availablePhpVersions = Services.detectPhpVersions()
             print("The following PHP versions were detected:")
             print(self.availablePhpVersions)
@@ -41,6 +42,12 @@ class AppDelegate: NSObject, NSApplicationDelegate {
         }
     }
     
+    func applicationWillTerminate(_ aNotification: Notification) {
+        // Insert code here to tear down your application
+    }
+    
+    // MARK: - UI related
+    
     func setStatusBarImage(version: String) {
         self.setStatusBar(image: ImageGenerator.generateImageForStatusBar(width: 32.0, text: version))
     }
@@ -52,24 +59,6 @@ class AppDelegate: NSObject, NSApplicationDelegate {
         }
     }
     
-    func applicationWillTerminate(_ aNotification: Notification) {
-        // Insert code here to tear down your application
-    }
-    
-    @objc func updatePhpVersionInStatusBar() {
-        self.version = PHPVersion()
-        if (self.busy) {
-            DispatchQueue.main.async {
-                self.setStatusBar(image: NSImage(named: NSImage.Name("StatusBarIcon"))!)
-            }
-        } else {
-            DispatchQueue.main.async {
-                self.setStatusBarImage(version: self.version!.short)
-            }
-        }
-        self.updateMenu()
-    }
-    
     func updateMenu() {
         DispatchQueue.global(qos: .userInitiated).async { [unowned self] in
             let menu = NSMenu()
@@ -78,13 +67,21 @@ class AppDelegate: NSObject, NSApplicationDelegate {
                 string = "You are running PHP \(self.version!.long)"
             }
             menu.addItem(NSMenuItem(title: string, action: nil, keyEquivalent: ""))
+            if (self.version != nil) {
+                // Actions
+                menu.addItem(NSMenuItem.separator())
+                menu.addItem(NSMenuItem(title: "Open php.ini in Finder", action: #selector(self.openActiveConfigFolder), keyEquivalent: ""))
+                // menu.addItem(NSMenuItem(title: "Restart PHP \(self.version!.short) service", action: #selector(self.restartPhp), keyEquivalent: ""))
+            }
             menu.addItem(NSMenuItem.separator())
             if (self.availablePhpVersions.count > 0 && !self.busy) {
-                for index in (0..<self.availablePhpVersions.count) {
+                var shortcutKey = 1
+                for index in (0..<self.availablePhpVersions.count).reversed() {
                     let version = self.availablePhpVersions[index]
                     let action = #selector(self.switchToPhpVersion(sender:))
-                    let menuItem = NSMenuItem(title: "Switch to PHP \(version)", action: (version == self.version?.short) ? nil : action, keyEquivalent: "\(index + 1)")
+                    let menuItem = NSMenuItem(title: "Switch to PHP \(version)", action: (version == self.version?.short) ? nil : action, keyEquivalent: "\(shortcutKey)")
                     menuItem.tag = index
+                    shortcutKey = shortcutKey + 1
                     menu.addItem(menuItem)
                 }
                 menu.addItem(NSMenuItem.separator())
@@ -101,11 +98,35 @@ class AppDelegate: NSObject, NSApplicationDelegate {
         }
     }
     
+    // MARK: - Callable via Obj-C (#selector)
+    
+    @objc func updatePhpVersionInStatusBar() {
+        self.version = PhpVersion()
+        if (self.busy) {
+            DispatchQueue.main.async {
+                self.setStatusBar(image: NSImage(named: NSImage.Name("StatusBarIcon"))!)
+            }
+        } else {
+            DispatchQueue.main.async {
+                self.setStatusBarImage(version: self.version!.short)
+            }
+        }
+        self.updateMenu()
+    }
+    
     @objc public func openAbout() {
         NSApplication.shared.activate(ignoringOtherApps: true)
         NSApplication.shared.orderFrontStandardAboutPanel()
     }
     
+    @objc public func openActiveConfigFolder() {
+        Services.openPhpConfigFolder(version: self.version!.short)
+    }
+    
+    @objc public func restartPhp() {
+        Services.restartPhp(version: self.version!.short)
+    }
+    
     @objc public func switchToPhpVersion(sender: AnyObject) {
         self.setStatusBar(image: NSImage(named: NSImage.Name("StatusBarIcon"))!)
         let index = sender.tag!

phpmon/Classes/PhpVersion.swift

@@ -8,21 +8,23 @@
 
 import Foundation
 
-class PHPVersion {
+class PhpVersion {
     
     var short : String = "???"
     var long : String = "???"
     
     init() {
-        // Get the info about the PHP installation
-        let output = Shell.execute(command: "php -v")
-        // Get everything before "(cli)" (PHP X.X.X (cli) ...)
-        var version = output.components(separatedBy: " (cli)")[0]
-        // Strip away the text before the version number
-        version = version.components(separatedBy: "PHP ")[1]
+        let version = Shell
+            // Get the version directly from PHP
+            .execute(command: "php -r 'print phpversion();'")
+            // also remove any colors
+            .replacingOccurrences(of: "\u{1b}(B\u{1b}[m", with: "")
+        
+        // That's the long version
         self.long = version
+        
         // Next up, let's strip away the minor version number
-        let segments = version.components(separatedBy: ".")
+        let segments = long.components(separatedBy: ".")
         // Get the first two elements
         self.short = segments[0...1].joined(separator: ".")
     }

phpmon/Helpers/Environment.swift

@@ -1,39 +0,0 @@
-//
-//  Environment.swift
-//  phpmon
-//
-//  Created by Nico Verbruggen on 12/06/2019.
-//  Copyright © 2019 Nico Verbruggen. All rights reserved.
-//
-
-import Foundation
-
-class Environment {
-    public static func performBootChecks()
-    {
-        if (!Shell.execute(command: "which php").contains("/usr/local/bin/php")) {
-            DispatchQueue.main.async {
-                Alert.present(
-                    messageText: "PHP is not correctly installed",
-                    informativeText: "You must install PHP via brew. Try running `which php` in Terminal, it should return `/usr/local/bin/php`. The app will not work correctly until you resolve this issue."
-                )
-            }
-        }
-        if (!Shell.execute(command: "ls /usr/local/opt | grep php@7.3").contains("php@7.3")) {
-            DispatchQueue.main.async {
-                Alert.present(
-                    messageText: "PHP 7.3 is not correctly installed",
-                    informativeText: "PHP 7.3 alias was not found in `/usr/local/opt`. The app will not work correctly until you resolve this issue."
-                )
-            }
-        }
-        if (!Shell.execute(command: "which valet").contains("/usr/local/bin/valet")) {
-            DispatchQueue.main.async {
-                Alert.present(
-                    messageText: "Laravel Valet is not correctly installed",
-                    informativeText: "You must install Valet via brew. Try running `which valet` in Terminal, it should return `/usr/local/bin/valet`. The app will not work correctly until you resolve this issue."
-                )
-            }
-        }
-    }
-}

phpmon/Info.plist

@@ -17,9 +17,11 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.0</string>
+	<string>1.1</string>
 	<key>CFBundleVersion</key>
-	<string>4</string>
+	<string>10</string>
+	<key>LSApplicationCategoryType</key>
+	<string>public.app-category.utilities</string>
 	<key>LSMinimumSystemVersion</key>
 	<string>$(MACOSX_DEPLOYMENT_TARGET)</string>
 	<key>LSUIElement</key>

phpmon/Startup/BootChecks.swift

@@ -0,0 +1,61 @@
+//
+//  Environment.swift
+//  phpmon
+//
+//  Created by Nico Verbruggen on 12/06/2019.
+//  Copyright © 2019 Nico Verbruggen. All rights reserved.
+//
+
+import Foundation
+
+class BootChecks {
+    
+    public static func perform()
+    {
+        self.presentAlertOnMainThreadIf(
+            !Shell.execute(command: "which php").contains("/usr/local/bin/php"),
+            messageText: "PHP is not correctly installed",
+            informativeText: "You must install PHP via brew. Try running `which php` in Terminal, it should return `/usr/local/bin/php`. The app will not work correctly until you resolve this issue."
+        )
+        
+        self.presentAlertOnMainThreadIf(
+            !Shell.execute(command: "ls /usr/local/opt | grep php@7.3").contains("php@7.3"),
+            messageText: "PHP 7.3 is not correctly installed",
+            informativeText: "PHP 7.3 alias was not found in `/usr/local/opt`. The app will not work correctly until you resolve this issue."
+        )
+        
+        self.presentAlertOnMainThreadIf(
+            !Shell.execute(command: "which valet").contains("/usr/local/bin/valet"),
+            messageText: "Laravel Valet is not correctly installed",
+            informativeText: "You must install Valet via brew. Try running `which valet` in Terminal, it should return `/usr/local/bin/valet`. The app will not work correctly until you resolve this issue."
+        )
+        
+        self.presentAlertOnMainThreadIf(
+            !Shell.execute(command: "cat /private/etc/sudoers.d/brew").contains("/usr/local/bin/brew"),
+            messageText: "Brew has not been added to sudoers.d",
+            informativeText: "You must run `sudo valet trust` to ensure Valet can start and stop services without having to use sudo every time. The app will not work correctly until you resolve this issue."
+        )
+        
+        self.presentAlertOnMainThreadIf(
+            !Shell.execute(command: "cat /private/etc/sudoers.d/valet").contains("/usr/local/bin/valet"),
+            messageText: "Valet has not been added to sudoers.d",
+            informativeText: "You must run `sudo valet trust` to ensure Valet can start and stop services without having to use sudo every time. The app will not work correctly until you resolve this issue."
+        )
+    }
+    
+    private static func presentAlertOnMainThreadIf(
+        _ condition: Bool,
+        messageText: String,
+        informativeText: String
+    )
+    {
+        if (condition) {
+            DispatchQueue.main.async {
+                Alert.present(
+                    messageText: messageText,
+                    informativeText: informativeText
+                )
+            }
+        }
+    }
+}

phpmon/Terminal/Services.swift

@@ -7,22 +7,17 @@
 //
 
 import Foundation
+import AppKit
 
 class Services {
     public static func mysqlIsRunning() -> Bool {
         let running = Shell.execute(command: "launchctl list | grep homebrew.mxcl.mysql")
-        if (running != "") {
-            return true
-        }
-        return false
+        return (running != "")
     }
     
     public static func nginxIsRunning() -> Bool {
         let running = Shell.execute(command: "launchctl list | grep homebrew.mxcl.nginx")
-        if (running != "") {
-            return true
-        }
-        return false
+        return (running != "")
     }
     
     public static func detectPhpVersions() -> [String] {
@@ -49,4 +44,13 @@ class Services {
             _ = Shell.execute(command: "valet use php@\(version)")
         }
     }
+    
+    public static func restartPhp(version: String) {
+        _ = Shell.execute(command: "brew services restart php@\(version)")
+    }
+    
+    public static func openPhpConfigFolder(version: String) {
+        let files = [NSURL(fileURLWithPath: "/usr/local/etc/php/\(version)/php.ini")];
+        NSWorkspace.shared.activateFileViewerSelecting(files as [URL]);
+    }
 }

phpmon/Terminal/Shell.swift

@@ -9,8 +9,8 @@
 import Cocoa
 
 class Shell {
-    public static func execute(command: String) -> String
-    {
+    
+    public static func execute(command: String) -> String {
         let task = Process()
         task.launchPath = "/bin/bash"
         task.arguments = ["--login", "-c", command]
@@ -20,7 +20,11 @@ class Shell {
         task.launch()
         
         let data = pipe.fileHandleForReading.readDataToEndOfFile()
-        let output: String = NSString(data: data, encoding: String.Encoding.utf8.rawValue)! as String
+        
+        let output: String = NSString(
+            data: data,
+            encoding: String.Encoding.utf8.rawValue
+        )! as String
         
         return output
     }

phpmon/View Controllers/ViewController.swift

@@ -0,0 +1,13 @@
+//
+//  ViewController.swift
+//  phpmon
+//
+//  Created by Nico Verbruggen on 11/06/2019.
+//  Copyright © 2019 Nico Verbruggen. All rights reserved.
+//
+
+import Cocoa
+
+class ViewController: NSViewController {
+
+}

phpmon/ViewController.swift

@@ -1,27 +0,0 @@
-//
-//  ViewController.swift
-//  phpmon
-//
-//  Created by Nico Verbruggen on 11/06/2019.
-//  Copyright © 2019 Nico Verbruggen. All rights reserved.
-//
-
-import Cocoa
-
-class ViewController: NSViewController {
-
-    override func viewDidLoad() {
-        super.viewDidLoad()
-
-        // Do any additional setup after loading the view.
-    }
-
-    override var representedObject: Any? {
-        didSet {
-        // Update the view, if already loaded.
-        }
-    }
-
-
-}
-