Bugfix/route resource prefixing

[cviebrock]

Here's an example routing situation:

Route::group(['prefix' => 'admin'], function()
{
    Route::model('post', '\Package\Model\Post');
    Route::resource('post', '\Package\PostController', ['as'=>'package']);
});

And currently, here are the resulting routes from artisan routes:

+--------+---------------------------------+----------------------------+---------------------------------+
| Domain | URI                             | Name                       | Action                          |
+--------+---------------------------------+----------------------------+---------------------------------+
|        | GET|HEAD admin/post             | package.admin.post.index   | \Package\PostController@index   |
|        | GET|HEAD admin/post/create      | package.admin.post.create  | \Package\PostController@create  |
|        | POST admin/post                 | package.admin.post.store   | \Package\PostController@store   |
|        | GET|HEAD admin/post/{post}      | package.admin.post.show    | \Package\PostController@show    |
|        | GET|HEAD admin/post/{post}/edit | package.admin.post.edit    | \Package\PostController@edit    |
|        | PUT admin/post/{post}           | package.admin.post.update  | \Package\PostController@update  |
|        | PATCH admin/post/{post}         |                            | \Package\PostController@update  |
|        | DELETE admin/post/{post}        | package.admin.post.destroy | \Package\PostController@destroy |
+--------+---------------------------------+----------------------------+---------------------------------+

With this patch, if you set the as option on the Route::resource() call, it will assume that that string is what you want to prefix your route names with, and not any of the route group names that are currently being added. This would solve several other issues (e.g. #1616) and updates the way #3000 solved it.

While there might be some BC issues, I don't see them as truly major. If you are passing 'as' to your Route::resource(), then I don't think you'd be expecting any prefixes anyway.

In my case, this is for a package that supplies routes where the group prefix is configurable. And yes, I could use action based routing (as has always been suggested), but I think this fixes a general issue where routes aren't being named in a predictable way when you think they should be. :)

Another solution would be to allow something like a no_group_prefix option to be passed, in which case my code would change from:

-       if ($prefix || count($this->groupStack) == 0)
+       if (isset($options['no_group_prefix']) || count($this->groupStack) == 0)

[cviebrock]

Oops ... PR to wrong branch. See #4507 instead.