[Bug Fix] Prevents DB Queries running when Artisan is running commands

[OwenMelbz]

Running the DB queries clashes with artisan cli when no database is available.

Meaning you cannot install/run artisan without setting up a database which makes deploying with systems like Envoyer, Codeship, Deploybot impossible.

[JoelESvensson]

My PR (#17) also fixes this

[OwenMelbz]

<3 @tabacitu

[tabacitu]

Hmm... @OwenMelbz this might have caused this issue: #21. Basically it won't publish the files anymore.

I think pulling these two lines out of the if (!\App::runningInConsole()) will fix it, but want to make sure it's still is ok with deploy systems. Could you please confirm? If so, I'll go ahead and make the fix.

Thanks, cheers!

[OwenMelbz]

Ah shoot, yes that would make sense!

Sorry I was a bit rushed trying to get the project up for a demo and didn't notice that bit :(

[tabacitu]

No worries, happens to all of us. Fixed in c82719f.

Cheers!

[Patabugen]

Is this addon not dependant on there being a Database? (since it stores the settings in the database?).

I'm probably missing a use case here - but making the library not work at all in Artisan so it can be run in some situations where someone's got no database but is still loading a library which depends on one seems like the lesser of the options.

Is there no way we can check for the presence of a database connection - and do nothing if we don't find one? Is it that common a problem to run Artisan without a DB where the HTTP version has one?