Unicode NoneType Problem with Initialization Function about phue HOT 6 CLOSED

👍 well not for hardcoding config but a way to manually specify the config file.

Basically change this (I've removed some comments to make it easier to read)

class Bridge(object):
    def __init__(self, ip=None, username=None):
        if os.access(os.getenv(USER_HOME), os.W_OK):
            self.config_file_path = os.path.join(
                os.getenv(USER_HOME), '.python_hue')
        else:
            self.config_file_path = os.path.join(os.getcwd(), '.python_hue')

becomes

class Bridge(object):
    def __init__(self, ip=None, username=None, config_file_path=None):
        if config_file_path and os.access(config_file_path, os.W_OK):
            self.config_file_path = config_file_path
        else:
            if os.access(os.getenv(USER_HOME), os.W_OK):
                self.config_file_path = os.path.join(
                    os.getenv(USER_HOME), '.python_hue')
            else:
                self.config_file_path = os.path.join(os.getcwd(), '.python_hue')

I'm sure it can be done better. but idea is add an optional config_file_path so it can be specified at runtime

from phue.

@studioimaginaire - any thoughts on this? Want to chime in?

from phue.

Will be back in town next week and will have a look !

from phue.

Thanks!

Jeremy Blum
Blum Idea Labs, LLC.
http://www.jeremyblum.com

Sent from my phone, please excuse the brevity.
On Jan 24, 2014 5:10 PM, "Nathanaël Lécaudé" [email protected]
wrote:

Will be back in town next week and will have a look !

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/21#issuecomment-33276543
.

from phue.

Hmmm,
You could also just pass the username to the bridge object :
b = Bridge('192.168.1.xx', 'username')
where username is the hex key.
Would that be a suitable workaround ?

from phue.

That seems like a perfectly reasonable approach to me. Do you think you may loop that into a future release?

from phue.