Removed app_home_location tilde in MySqlWorkbenchCommand.php

terminus-plugin-project / terminus-pancakes-plugin

Terminus Plugin to open Pantheon Site Databases in your Favorite SQL Client

MIT License

26 stars 20 forks source link

Removed app_home_location tilde in MySqlWorkbenchCommand.php #8

Closed uberhacker closed 8 years ago

uberhacker commented 8 years ago

Switched to getenv('HOME') instead of tilde to correct the path
Added logic to determine the sftp port since git_port is not available in the connection info for test and live sites
Changed UseSSL from 2 to 1

Believe it or not, the tilde causes problems with file operations. I've seen the same thing happen in Mac OS also so we better add back in getenv('HOME') in the paths.

derimagia commented 8 years ago

Weird, tilde worked fine for me on mac, never had any issues with it but might as well just use the variable. Looks good