Skip to content

[Maybe BUG] Laravel 5 softDeletes Model cause blank white page response if table don't have deleted_at column #3514

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 171 commits into from
Closed

[Maybe BUG] Laravel 5 softDeletes Model cause blank white page response if table don't have deleted_at column #3514

wants to merge 171 commits into from

Conversation

darkcyber
Copy link

Hi,

My Options table contain id, name, value, autoload columns.

My routes.php

<?php
Route::get('/settings', 'SettingsController@index');

Here is my SettingsController.php

<?php namespace App\Http\Controllers\Admin;

use App\Http\Requests;
use App\Http\Controllers\Controller;
use App\Option;

class SettingsController extends Controller {

    public function index()
    {
        $options = Option::all();
        return dd($options);
    }

Here is my Option.php model

<?php namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;

/**
* 
*/
class Option extends Model
{
    use SoftDeletes;
    protected $fillable = ['id', 'name', 'value', 'autoload'];
    protected $table = 'options';
}

If I visit myproject.dev/settings it will give me blank white page without any error, try to delete views cache at storage folder didn't work.

Try with artisan tinker give me some idea

use App\Option;
dd(Option::all());

It give me Illuminate\Database\QueryException with message 'SQLSTATE[42S22]: Column not found: 1054 Unknown column 'coeg_options.deleted_at' in 'where clause' (SQL: select * fromcoeg_optionswherecoeg_options.deleted_atis null)

Then I add deleted_at column at my table, finally it work and I didn't get blank white pages anymore.

Maybe will be good if Laravel check deleted_at column for model which use SoftDeletes and throw an error if deleted_at column not found, just like artisan tinker error above.

taylorotwell and others added 30 commits February 4, 2015 08:17
@taylorotwell
upgrade version.
6184206
@taylorotwell
upgrade version.
9da5d30
@taylorotwell
Merge branch 'master' into develop
b2c2962
@taylorotwell
Merge branch 'master' into develop
38aae74
@taylorotwell
Merge branch 'master' into develop
da6fc17
@taylorotwell
Merge branch 'master' into develop
d75ed33
@GrahamCampbell
Fixed 5.1 installation
c170c76
@taylorotwell
Simplify authentication. Remove service.
d5f285b
@taylorotwell
Tweaks.
d2d9c79
@taylorotwell
Tweak some jobs and names.
1500a18
@taylorotwell
Merge pull request #3267 from GrahamForks/fix
01cf5f1 
Fixed 5.1 installation
@taylorotwell
Merge branch 'master' into develop
2f17860
@taylorotwell
Merge branch 'master' into develop
76d4c11
@taylorotwell
PSR-2 for app.
f424b87
@taylorotwell
Tweaking a few things.
cc2139a
@taylorotwell
Tweak a few things.
4c78958
@taylorotwell
Merge branch 'master' into develop
e590bf2
@taylorotwell
Connection setting in Redis queue.
eacb058
@marcus-herrmann
Use "for" attribute on labels in auth views
4f88bcb
@taylorotwell
Merge pull request #3279 from marcus-herrmann/develop
73e1c71 
Use "for" attribute on labels in auth views
@taylorotwell
fix conflicts.
338f5f1
@taylorotwell
fix bad merge.
5e71bca
@taylorotwell
conflicts.
5e71cc5
@taylorotwell
Merge branch 'master' into develop
055edad
@taylorotwell
Just use helper function in middleware.
c9c0380
@taylorotwell
Some spacing.
8909e75
@sineld
support for unencrypted mail server usage like mailcatcher.me
fad5c6e
@taylorotwell
Merge pull request #3304 from sineld/develop
ce5bcb2 
support for unencrypted mail server usage like mailcatcher.me
@taylorotwell
fix conflicts.
5644e8e
@taylorotwell
Remove auth scaffolding to make it opt-in.
5e11f87
jonathantorres and others added 28 commits July 17, 2015 23:27
@jonathantorres
Newline.
fef3aa0
@taylorotwell
Merge pull request #3459 from jonathantorres/new-line
9b359ba 
Newline on .env.example
Sorted by alphabetic.
191c687 
Sorted by alphabetic.
@taylorotwell
Merge pull request #3465 from jumilla/patch-1
0cf34a3 
Sorted by alphabetic.
@taylorotwell
Convert some variables to env variables in services.php.
8061c21
@andrewhood125
Ignore Homestead.json
12386cf
@taylorotwell
Merge pull request #3467 from andrewhood125/homestead-json
9d82fc6 
Ignore Homestead.json
Bump Laravel Elixir version
64e1cf9
@taylorotwell
Merge pull request #3468 from laracasts/patch-1
51299e7 
Bump Laravel Elixir version
@taylorotwell
working on exception handling for model not found.
da7376d
@taylorotwell
Merge remote-tracking branch 'origin/master'
f478406
@taylorotwell
fix status code
395b69b
@taylorotwell
fix incorrect method name in comment.
f047f80
@bmitch
Error message for JSON validation
33272b3
@taylorotwell
Merge pull request #3480 from bmitch/JsonErrorMsg
2ddeae9 
Error message for JSON validation
@Zarthus
Use https rather than // for google fonts in default views.
b8790d5 
Ensure the default views always use HTTPS for the loading of fonts. This is overall a good practise to follow.
@taylorotwell
Merge pull request #3484 from Zarthus/patch/views-use-https
f40e72f 
Use https rather than // for google fonts in default views.
@taylorotwell
acl changes
c82c5ec
@taylorotwell
Merge remote-tracking branch 'origin/master'
9bb3c9f
@taylorotwell
Define HasAbilities trait on default user.
72a10c6
@taylorotwell
Implement interface.
420e813
@taylorotwell
Default auth service provider.
7d4b5d7
@GrahamCampbell @StyleCIBot
Applied fixes from StyleCI
35654d7
@GrahamCampbell
Merge pull request #3491 from laravel/analysis-zObJlq
716e652 
Applied fixes from StyleCI
@pedes42
Replace storage_path with database_path helper
879bc14 
According to the docs, the sqlite database should be located within the "database" folder.
@taylorotwell
Merge pull request #3500 from pedes42/master
24575f4 
Replace storage_path with database_path helper
@crynobone
Use $this instead of parent.
50357d8 
Signed-off-by: crynobone <crynobone@gmail.com>
@taylorotwell
Merge pull request #3501 from crynobone/feature/use-this
b0a22fc 
[5.1] Use $this instead of parent.
@asoorm
Copy link

asoorm commented Oct 6, 2015

Don't use softDeletes in your model, and Laravel will know not to look
at the deleted_at column

On 06/10/2015 10:26, Dark Cyber wrote:

Hi,

My |Options| table contain |id, name, value, autoload| columns.

My |routes.php|

|<?php Route::get('/settings', 'SettingsController@index'); |

Here is my |SettingsController.php|

|<?php namespace App\Http\Controllers\Admin; use App\Http\Requests; use
App\Http\Controllers\Controller; use App\Option; class
SettingsController extends Controller { public function index() {
$options = Option::all(); return dd($options); } |

Here is my |Option.php| model

|<?php namespace App; use Illuminate\Database\Eloquent\Model; use
Illuminate\Database\Eloquent\SoftDeletes; /** * */ class Option
extends Model { use SoftDeletes; protected $fillable = ['id', 'name',
'value', 'autoload']; protected $table = 'options'; } |

If I visit |myproject.dev/settings| it will give me blank white page
without any error, try to delete views cache at storage folder didn't
work.

Try with artisan tinker give me some idea

|use App\Option; dd(Option::all()); |

It give me |Illuminate\Database\QueryException with message
'SQLSTATE[42S22]: Column not found: 1054 Unknown column
'coeg_options.deleted_at' in 'where clause' (SQL: select *
from|coeg_options|where|coeg_options|.|deleted_at|is null)|

Then I add |deleted_at| column at my table, finally it work and I
didn't get blank white pages anymore.

    You can view, comment on, or merge this pull request online at:

#3514

    Commit Summary

—
Reply to this email directly or view it on GitHub
#3514.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.