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.Views supports to analyze your views for certain common configuration problems,
though the actual output looks broken see attached screenshot.
I guess this can be solved using some css or adding additional classes.
In the attachment you can find a view, which fails on analyze, so it's easy to reproduce.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 1533486.patch | 426 bytes | dawehner |
| #3 | 1533486.patch | 425 bytes | Anonymous (not verified) |
| #2 | 1533486.patch | 425 bytes | dawehner |
| #1 | views-1533486-analysis_warning_messages.patch | 1.6 KB | Anonymous (not verified) |
| analyze.view_.txt | 1.49 KB | dawehner | |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedI fixed this issue by adding a "messages" class in two places.
Comment #2
dawehnerThanks for the patch, it really helps, though the approach seems to be a bit unflexible.
Analyzing of views can be implemented by any kind of module, so it should be added to all of them.
what about a patch like this? It would be cool if you test the patch.
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedI like your method better but the class is messages not message. I just updated your patch and reposted.
Comment #5
dawehnerOh i see, this patch should work again.
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commentedThanks. Thought I could just add the s. Lesson learned.
That one works.
Comment #7
dawehnerThanks again, committed the patch to 7.x-3.x
You have now a commit in views: http://drupal.org/node/38878/commits