👍

I'm wondering if it would be safest to tag Laravel framework before merging this in case people clone this repo and then run composer install and get the current tagged version which doesn't have those changes?

Yeah that's what I meant, a new laravel/framework tag (not laravel/laravel).

Refs laravel/framework#12827 which is what this PR makes use of.

I guess, now laravel/framework v5.2.25 is tagged, this can be merged?
Or do you want to back-port to 5.1 first?

This works perfectly btw, updated my composer.json file :-)

str_replace_array not support for array field ,
"permissions" is a text in "roles" table and it has been set $casts = [ 'permissions'=>'array' ] in Role model , error message:

in helpers.php line 740
at HandleExceptions->handleError('2', 'preg_replace(): Parameter mismatch, pattern is a string while replacement is an array', '/mnt/code/htdocs/xiwo/www/vendor/illuminate/support/helpers.php', '740', array('search' => '?', 'replace' => array('Admin', '1', array('5', '6', '7', '8'), '2016-03-27 08:28:53', '3'), 'subject' => 'update roles set name = Admin, type = 1, permissions = ?, updated_at = ? where id = ?', 'value' => array('5', '6', '7', '8')))
at preg_replace('/?/', array('5', '6', '7', '8'), 'update roles set name = Admin, type = 1, permissions = ?, updated_at = ? where id = ?', '1') in helpers.php line 740
at str_replace_array('?', array('Admin', '1', array('5', '6', '7', '8'), '2016-03-27 08:28:53', '3'), 'update roles set name = ?, type = ?, permissions = ?, updated_at = ? where id = ?') in QueryException.php line 56
at QueryException->formatMessage('update roles set name = ?, type = ?, permissions = ?, updated_at = ? where id = ?', array('Admin', '1', array('5', '6', '7', '8'), '2016-03-27 08:28:53', '3'), object(ErrorException)) in QueryException.php line 39

Above issue by reavenlin is unrelated, and I'm pretty sure is due to incorrect use of query builder.