Merge pull request #1429 from braceyourself/master
Use config helper instead of env in application code
This commit is contained in:
commit
cd887e8740
@ -23,11 +23,11 @@ class Kernel extends ConsoleKernel
|
||||
protected function schedule(Schedule $schedule)
|
||||
{
|
||||
// Not installed yet
|
||||
if (!env('APP_INSTALLED')) {
|
||||
if (!config('app.installed')) {
|
||||
return;
|
||||
}
|
||||
|
||||
$schedule_time = env('APP_SCHEDULE_TIME', '09:00');
|
||||
$schedule_time = config('app.schedule_time');
|
||||
|
||||
$schedule->command('reminder:invoice')->dailyAt($schedule_time);
|
||||
$schedule->command('reminder:bill')->dailyAt($schedule_time);
|
||||
|
@ -54,7 +54,7 @@ class Handler extends ExceptionHandler
|
||||
*/
|
||||
public function render($request, Throwable $exception)
|
||||
{
|
||||
if (env('APP_DEBUG') === false) {
|
||||
if (config('app.debug') === false) {
|
||||
return $this->handleExceptions($request, $exception);
|
||||
}
|
||||
|
||||
|
@ -115,7 +115,7 @@ class Login extends Controller
|
||||
auth()->logout();
|
||||
|
||||
// Session destroy is required if stored in database
|
||||
if (env('SESSION_DRIVER') == 'database') {
|
||||
if (config('session.driver') == 'database') {
|
||||
$request = app('Illuminate\Http\Request');
|
||||
$request->session()->getHandler()->destroy($request->session()->getId());
|
||||
}
|
||||
|
@ -27,8 +27,10 @@ class Database extends Controller
|
||||
*/
|
||||
public function store(Request $request)
|
||||
{
|
||||
$connection = config('database.default','mysql');
|
||||
|
||||
$host = $request['hostname'];
|
||||
$port = env('DB_PORT', '3306');
|
||||
$port = config("database.connections.$connection.port", '3306');
|
||||
$database = $request['database'];
|
||||
$username = $request['username'];
|
||||
$password = $request['password'];
|
||||
|
@ -19,7 +19,7 @@ class Notifications
|
||||
public function compose(View $view)
|
||||
{
|
||||
// No need to add suggestions in console
|
||||
if (app()->runningInConsole() || !env('APP_INSTALLED') || !user()) {
|
||||
if (app()->runningInConsole() || !config('app.installed') || !user()) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -20,7 +20,7 @@ class Suggestions
|
||||
public function compose(View $view)
|
||||
{
|
||||
// No need to add suggestions in console
|
||||
if (app()->runningInConsole() || !env('APP_INSTALLED')) {
|
||||
if (app()->runningInConsole() || !config('app.installed')) {
|
||||
return;
|
||||
}
|
||||
|
||||
@ -31,10 +31,10 @@ class Suggestions
|
||||
|
||||
if ($path) {
|
||||
$suggestions = $this->getSuggestions($path);
|
||||
|
||||
|
||||
if ($suggestions) {
|
||||
$suggestion_modules = $suggestions->modules;
|
||||
|
||||
|
||||
foreach ($suggestion_modules as $key => $module) {
|
||||
$installed = Module::where('company_id', session('company_id'))->where('alias', $module->alias)->first();
|
||||
|
||||
|
@ -26,7 +26,7 @@ class Version207 extends Listener
|
||||
|
||||
// Update .env file
|
||||
Installer::updateEnv([
|
||||
'MAIL_MAILER' => env('MAIL_DRIVER'),
|
||||
'MAIL_MAILER' => env('MAIL_DRIVER', config('mail.default')),
|
||||
]);
|
||||
}
|
||||
}
|
||||
|
@ -31,11 +31,11 @@ class App extends Provider
|
||||
*/
|
||||
public function register()
|
||||
{
|
||||
if (env('APP_INSTALLED') && env('APP_DEBUG')) {
|
||||
if (config('app.installed') && config('app.debug')) {
|
||||
$this->app->register(\Barryvdh\Debugbar\ServiceProvider::class);
|
||||
}
|
||||
|
||||
if (env('APP_ENV') !== 'production') {
|
||||
if (config('app.env') !== 'production') {
|
||||
$this->app->register(\Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class);
|
||||
}
|
||||
}
|
||||
|
@ -17,7 +17,7 @@ trait DateTime
|
||||
$default = 'd M Y';
|
||||
|
||||
// Make sure it's installed
|
||||
if (!env('APP_INSTALLED') && (env('APP_ENV') !== 'testing')) {
|
||||
if (!config('app.installed') && (config('app.env') !== 'testing')) {
|
||||
return $default;
|
||||
}
|
||||
|
||||
|
@ -33,7 +33,7 @@ class Info
|
||||
|
||||
public static function mysqlVersion()
|
||||
{
|
||||
if (env('DB_CONNECTION') === 'mysql') {
|
||||
if (config('database.default') === 'mysql') {
|
||||
return DB::selectOne('select version() as mversion')->mversion;
|
||||
}
|
||||
|
||||
|
@ -183,8 +183,8 @@ class Installer
|
||||
'database' => $database,
|
||||
'username' => $username,
|
||||
'password' => $password,
|
||||
'driver' => env('DB_CONNECTION', 'mysql'),
|
||||
'charset' => env('DB_CHARSET', 'utf8mb4'),
|
||||
'driver' => $connection = config('database.default', 'mysql'),
|
||||
'charset' => config("database.connections.$connection.charset", 'utf8mb4'),
|
||||
]);
|
||||
|
||||
try {
|
||||
@ -213,7 +213,7 @@ class Installer
|
||||
'DB_PREFIX' => $prefix,
|
||||
]);
|
||||
|
||||
$con = env('DB_CONNECTION', 'mysql');
|
||||
$con = config('database.default', 'mysql');
|
||||
|
||||
// Change current connection
|
||||
$db = Config::get('database.connections.' . $con);
|
||||
@ -309,6 +309,8 @@ class Installer
|
||||
|
||||
file_put_contents(base_path('.env'), $env);
|
||||
|
||||
Artisan::call('config:cache');
|
||||
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
@ -14,6 +14,10 @@ return [
|
||||
|
||||
'name' => env('APP_NAME', 'Akaunting'),
|
||||
|
||||
'installed' => env('APP_INSTALLED', false),
|
||||
|
||||
'schedule_time' => env('APP_SCHEDULE_TIME', '9:00'),
|
||||
|
||||
/*
|
||||
|--------------------------------------------------------------------------
|
||||
| Application Environment
|
||||
|
@ -33,7 +33,7 @@
|
||||
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
@ -28,7 +28,7 @@
|
||||
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
@ -31,7 +31,7 @@
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_home = '{{ url("apps/categories") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
@ -30,7 +30,7 @@
|
||||
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
@ -30,7 +30,7 @@
|
||||
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
@ -30,7 +30,7 @@
|
||||
|
||||
<script type="text/javascript"><!--
|
||||
var url = '{{ url("/") }}';
|
||||
var app_url = '{{ env("APP_URL") }}';
|
||||
var app_url = '{{ config("app.url") }}';
|
||||
//--></script>
|
||||
|
||||
@stack('js')
|
||||
|
Loading…
x
Reference in New Issue
Block a user