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

Add method to get route by language #3188

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Karmalakas
Copy link
Contributor

@Karmalakas Karmalakas commented Jan 30, 2021

Just an idea on how redirect problem might be fixed maybe. Or at least partially.
Although this doesn't work - couldn't figure out how to get parent. Without the parent, route() returns just a home route.
Maybe someone could take this over, or tell me the approach is completely wrong :) But that issue really needs to be fixed. Otherwise I'll have to search for alternatives and I really like Grav, so it would be sad if I couldn't publish my page and had to ditch a few weeks work :(

Also Page::init() calls $this->id($this->modified() . md5($this->filePath())), but it gets overwritten, because $this->id doesn't exist yet, so added a check in Page::id()

Just an idea on how [redirect problem](https://github.com/getgrav/grav/issues/3180) might be fixed maybe. Or at  least partially.
Although this doesn't work - couldn't figure out how to get parent. Without the parent, `route()` returns just a home route.
Maybe someone could take this over, or tell me the approach is completely wrong :) But that issue really needs to be fixed. Otherwise I'll have to search for alternatives and I really like Grav otherwise, so it would be sad if I couldn't publish my page and had to ditch a few weeks work :(

Also `Page::init()` calls `$this->id($this->modified() . md5($this->filePath()))`, but it gets overwritten, because `$this->id`  doesn't exist yet, so added a check in `Page::id()`
@OleVik
Copy link
Contributor

OleVik commented Jan 30, 2021

Parent needs to be instantiated, no? I would assume that this goes into the comprehensive work on improving multi-language, @ricardo118? Both pull requests and related issues address deficiencies in current language-handling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants