Problem/Motivation
Currently, the routes to rewrite can only be configured by developers by editing the route_path_rewrite.settings.yml file.
Proposed resolution
Create a module settings form with routes to rewrite.
Remaining tasks
- Create a module settings form with routes to rewrite
- Update README.md and project page
- Commit
User interface changes
Module settings form with routes to rewrite
API changes
None.
Data model changes
None.
Release notes snippet
Module settings form with routes to rewrite added
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | add-setting-form-3026914-5.patch | 9.39 KB | guilherme-lima-almeida |
| #4 | Added-settings-form-3026914-3.patch | 7.32 KB | Andrey Proh |
| #3 | Added-settings-form-3026914-3.patch | 7.37 KB | Andrey Proh |
Issue fork route_path_rewrite-3026914
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 8.x-1.x
changes, plain diff MR !3
- 3026914-make-routes-to
changes, plain diff MR !2
Comments
Comment #2
manuel.adanYes, this is a must. There are similar use cases in core and contrib that can be used as reference.
Comment #3
Andrey Proh commentedPlease review.
Comment #4
Andrey Proh commentedFixed notice "Notice: Undefined variable: keys in Drupal\…"
Comment #5
guilherme-lima-almeidaHi Andrey Proh,
I added dependency injection and added the validateForm method to check if the routes the user insert exists.
Comment #9
milos33 commentedCorrection to use existing routes from configuration.
Comment #10
milos33 commentedPatch fix, for 2.0.0 using existing config.
Comment #11
rdamjanov commentedRemoved from display because it still needs some work.
Comment #13
rdamjanov commentedI am closing this issue because we have recommended release 2.0.1 - which contains this feature.