Skip to content
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.

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

Admin Plugin change for multisite setup #1674

Closed
trianity opened this issue May 7, 2019 · 1 comment
Closed

Admin Plugin change for multisite setup #1674

trianity opened this issue May 7, 2019 · 1 comment

Comments

@trianity
Copy link

trianity commented May 7, 2019

This is a suggestion for the minor change in the
file: user\plugins\admin\admin.php
About line 470 there is a block:
// If not provided by Admin, lookup pages added by other plugins
Inside the foreach loop there is a line:

$path = $locator->findResource("user://plugins/{$plugin->name}/admin/pages/{$self->template}.md");

My suggestion is change the stream from user:// to plugins:// like this

$path = $locator->findResource("plugins://{$plugin->name}/admin/pages/{$self->template}.md");

The reason is, if the multisite setup uses the changed user stream, like provided below, then the admin plugin will not find any page provided by the other plugins.

'streams' => [
        'schemes' => [
            'user' => [
               'type' => 'ReadOnlyStream',
               'prefixes' => [
                   '' => ["user/{$folder}"],
               ]
            ],

This change was tested in my multisite setup with success.

@rhukster
Copy link
Member

rhukster commented May 8, 2019

Agreed, done...

@rhukster rhukster closed this as completed May 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants