handle URLs on plugins that include only handler name

[cash]

Some are serving 404 pages if you hit ​http://example.org/<handler>/. I expect that would serve the same content as ​http://example.org/<handler>/all

[sembrestels]

Do you refer to add this code

if (empty($page[0])) {
    forward("$handler/all");
}

in all page handlers that is missing, or to change the core page_handler function?

[sembrestels]

There is a related issue: 403 Forbidden in ​http://your.elgg/pages

This is because we have a pages folder in the elgg root.

[cash]

We don't want to change the core page handler. Each plugin should handle it.

Maybe this instead:

if (empty($page[0])) {
    $page[0] = 'all';
}

[sembrestels]

PR: ​https://github.com/Elgg/Elgg/pull/318

[cash]

Thanks, Sem

[Sem]

Fixes #4687. Handle URLs on plugins that include only handler name.

Changeset: d6953c9955b2b9e3c245936e448e7be41058bb9b

[Cash Costello]

Merge pull request #318 from sembrestels/handle-all

Fixes #4687. Handle URLs on plugins that include only handler name.

Changeset: 901cb69ba0b40d247bea5b0e1b4e4588a86800c1