Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#5 | interdiff.txt | 3.17 KB | Gábor Hojtsy |
#5 | 2257455-schema-check-5.patch | 5.43 KB | Gábor Hojtsy |
#4 | 2257455-schema-check-4.patch | 4.32 KB | amitgoyal |
#1 | 2257455-schema-check-1.patch | 3.19 KB | vijaycs85 |
Comments
Comment #1
vijaycs85patch fixes below issues:
1. Fix the controller to use hasSchema() method.
2. Adding help text to say that the overview page displays only config with schema.
3. Adding title to overview page.
4. Adding form support for color_hex data type from #2245729: Add missing configuration schema in Color component
Comment #2
vijaycs85We can add test for all 4 items in #1
Comment #3
Gábor HojtsyNow that I look at this I think this would be be a better debugging tool if the schema-less items would be displayed also but not linked (no operations, or "No schema available" printed in place of operations). At least you would know those are there and what is the problem if you cannot inspect them. Otherwise looks like good simplification.
Comment #4
amitgoyal CreditAttribution: amitgoyal commented@Gábor - Please see updated patch. Schema-less items would be displayed but not linked.
Comment #5
Gábor Hojtsy- Fixed hook_help text to give more help.
- Fixed hook_help signature so it works with current request based system.
- Fixed listing of no-schema items to allow review of raw data.
- Improved raw data callback so it can work without a schema depicting missing schema as NULL.
- Committing this one in the interest of marching forward helping core find places where it has no schema. We should add more tests later on anyway :) #2253749: Add test for all route controllers should add tests.
Comment #7
vijaycs85thanks @Gábor Hojtsy.