Problem/Motivation
The module's UI and documentation can be improved:
- improve menu appearance
- use more consistent UI labels
- add more documentation
- use README template
- remove "entity" from UI labels
- use "path" consistent
- consistent route
- use views for overview- consistent casing
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | Screenshot From 2026-08-28 11-14-30.png | 29.09 KB | arjenk |
Issue fork path_file-3538124
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:
Comments
Comment #2
batigolixComment #4
batigolixComment #5
batigolixComment #6
batigolixComment #7
batigolixComment #8
batigolixComment #9
batigolixComment #10
batigolixComment #11
batigolixComment #12
arjenk commentedReviewed and manually tested MR !16. Clear improvements, the move to /admin/content makes sense.
Some minor findings:
'#default_value' => $config->get('allowed_extensions') ?: 'pdf doc docx txt',. If an admin never opens the settings form, path_file.settings:allowed_extensions stays empty. An empty file_extensions means no extension restriction (any file type uploadable). Add a config/install/path_file.settings.yml with allowed_extensions: 'pdf doc docx txt'.'#element_validate' => [['Drupal\file\Plugin\Field\FieldType\FileItem', 'validateExtensions']]). Reusing it also allows.and_(e.g.tar.gz), comma separatorsComment #13
batigolixComment #16
batigolixI fixed the mentioned problems. This can be reviewed again
Comment #17
arjenk commentedReviewed and tested locally; tests are green, also with the current 2.1.x merged. The mentioned problems are solved.
One left over occurrence in the UI of 'entity':

Otherwise all good.
Comment #18
batigolixTop! Many Thank