diff --git a/app/Http/Controllers/AppController.php b/app/Http/Controllers/AppController.php index 0ea384cf29aa..4c55daeb868c 100644 --- a/app/Http/Controllers/AppController.php +++ b/app/Http/Controllers/AppController.php @@ -13,6 +13,7 @@ use Session; use Cookie; use Response; use App\Models\User; +use App\Models\Account; use App\Ninja\Mailers\Mailer; use App\Ninja\Repositories\AccountRepository; use Redirect; @@ -32,18 +33,12 @@ class AppController extends BaseController public function showSetup() { - if (Utils::isNinja() || Utils::isDatabaseSetup()) { + if (Utils::isNinja() || (Utils::isDatabaseSetup() && Account::count() > 0)) { return Redirect::to('/'); } $view = View::make('setup'); - /* - $cookie = Cookie::forget('ninja_session', '/', 'www.ninja.dev'); - Cookie::queue($cookie); - return Response::make($view)->withCookie($cookie); - */ - return Response::make($view); } diff --git a/app/Http/Controllers/HomeController.php b/app/Http/Controllers/HomeController.php index 1db0b1698224..49ebc9343c2c 100644 --- a/app/Http/Controllers/HomeController.php +++ b/app/Http/Controllers/HomeController.php @@ -27,10 +27,8 @@ class HomeController extends BaseController { Session::reflash(); - if (!Utils::isDatabaseSetup()) { + if (!Utils::isNinja() && (!Utils::isDatabaseSetup() || Account::count() == 0)) { return Redirect::to('/setup'); - } elseif (Account::count() == 0) { - return Redirect::to('/invoice_now'); } elseif (Auth::check()) { return Redirect::to('/dashboard'); } else { diff --git a/app/Ninja/Mailers/Mailer.php b/app/Ninja/Mailers/Mailer.php index 21f82009b749..9e32e4ee2fbd 100644 --- a/app/Ninja/Mailers/Mailer.php +++ b/app/Ninja/Mailers/Mailer.php @@ -15,11 +15,7 @@ class Mailer Mail::send($views, $data, function ($message) use ($toEmail, $fromEmail, $fromName, $subject, $data) { $replyEmail = $fromEmail; - - // http://stackoverflow.com/questions/2421234/gmail-appearing-to-ignore-reply-to - if (Utils::isNinja() && $toEmail != CONTACT_EMAIL) { - $fromEmail = NINJA_FROM_EMAIL; - } + $fromEmail = NINJA_FROM_EMAIL; if(isset($data['invoice_id'])) { $invoice = Invoice::with('account')->where('id', '=', $data['invoice_id'])->get()->first(); diff --git a/resources/lang/en/texts.php b/resources/lang/en/texts.php index bb940d5f5a37..2d600d31f970 100644 --- a/resources/lang/en/texts.php +++ b/resources/lang/en/texts.php @@ -704,4 +704,6 @@ return array( 'login' => 'Login', 'or' => 'or', + 'email_error' => 'There was a problem sending the email', + ); diff --git a/resources/views/setup.blade.php b/resources/views/setup.blade.php index e65ebae5d7f7..c0f2daab0d58 100644 --- a/resources/views/setup.blade.php +++ b/resources/views/setup.blade.php @@ -6,10 +6,11 @@ + @@ -64,7 +65,7 @@ FLUSH PRIVILEGES;

Application Settings

- {!! Former::text('app[url]')->label('URL')->value(Request::root()) !!} + {!! Former::text('app[url]')->label('URL')->value(isset($_ENV['APP_URL']) ? $_ENV['APP_URL'] : Request::root()) !!}
@@ -73,12 +74,17 @@ FLUSH PRIVILEGES;

Database Connection

- {!! Former::select('database[default]')->label('Driver')->options(['mysql' => 'MySQL', 'pgsql' => 'PostgreSQL', 'sqlite' => 'SQLite']) !!} - {!! Former::text('database[type][host]')->label('Host')->value('localhost') !!} - {!! Former::text('database[type][database]')->label('Database')->value('ninja') !!} - {!! Former::text('database[type][username]')->label('Username')->value('ninja') !!} - {!! Former::password('database[type][password]')->label('Password')->value('ninja') !!} - {!! Former::actions( Button::normal('Test connection')->withAttributes(['onclick' => 'testDatabase()']), '  ' ) !!} + {!! Former::select('database[default]')->label('Driver')->options(['mysql' => 'MySQL', 'pgsql' => 'PostgreSQL', 'sqlite' => 'SQLite']) + ->value(isset($_ENV['DB_TYPE']) ? $_ENV['DB_TYPE'] : 'mysql') !!} + {!! Former::text('database[type][host]')->label('Host')->value('localhost') + ->value(isset($_ENV['DB_HOST']) ? $_ENV['DB_HOST'] : '') !!} + {!! Former::text('database[type][database]')->label('Database')->value('ninja') + ->value(isset($_ENV['DB_DATABASE']) ? $_ENV['DB_DATABASE'] : '') !!} + {!! Former::text('database[type][username]')->label('Username')->value('ninja') + ->value(isset($_ENV['DB_USERNAME']) ? $_ENV['DB_USERNAME'] : '') !!} + {!! Former::password('database[type][password]')->label('Password')->value('ninja') + ->value(isset($_ENV['DB_PASSWORD']) ? $_ENV['DB_PASSWORD'] : '') !!} + {!! Former::actions( Button::primary('Test connection')->small()->withAttributes(['onclick' => 'testDatabase()']), '  ' ) !!}
@@ -95,7 +101,7 @@ FLUSH PRIVILEGES; {!! Former::text('mail[from][name]')->label('From Name') !!} {!! Former::text('mail[username]')->label('Email') !!} {!! Former::password('mail[password]')->label('Password') !!} - {!! Former::actions( Button::normal('Send test email')->withAttributes(['onclick' => 'testMail()']), '  ' ) !!} + {!! Former::actions( Button::primary('Send test email')->small()->withAttributes(['onclick' => 'testMail()']), '  ' ) !!} @@ -113,7 +119,7 @@ FLUSH PRIVILEGES; {!! Former::checkbox('terms_checkbox')->label(' ')->text(trans('texts.agree_to_terms', ['terms' => ''.trans('texts.terms_of_service').''])) !!} - {!! Former::actions( Button::primary('Submit')->submit() ) !!} + {!! Former::actions( Button::primary('Submit')->large()->submit() ) !!} {!! Former::close() !!}