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.
Forena's 7.x-4x-beta3 version now also displays some forena related info on Drupal's status report (within /admin/reports/status). However some minor refinements for the display of these messages seem appropriate, i.e. related to:
- the info about the Forena report directory.
- the info about the SVGGraph support (and similarly the dataTables support, and the PDF support libraries).
The additional comments in this issue provide some more details about the suggested refinements.
Comments
Comment #1
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedRegarding the info about the Forena report directory:
Refer to attached patch file to implement these refinements.
Comment #2
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedRegarding the info about the SVGGraph support (and similarly the dataTables support):
Similar remark for the dataTables support.
Refer to attached patch file to implement these suggested refinements.
Comment #3
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedComment #4
metzlerd CreditAttribution: metzlerd commentedIt's unclear from the existing message what I'm supposed to do. Would it be possible to add a link in that message to the appropriate configuration page so that the user is taken to the page where they need to confirm the configuration of the reports directory?
Comment #5
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedDavid, here is a patch to replace my prior patch in #1, containing the corrections you asked about in #4. This patch is also re-rolled against current dev version (instead of beta3 before), and using a path relative to the forena dir. Attached are also 2 new png files to illustrate my newest test results, and which replace the Rpt-Dir-Refined-Error-Message.png from my #1 (the other pngs in #1 are unchanged).
If you agree, let me know if you want me to apply it to the dev version (alternatively go ahead to get it applied).
Comment #6
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedDavid, here is a patch to replace my prior patch in #2. This patch is re-rolled against current dev version (instead of beta3 before), and using a path relative to the forena dir. No other (functional) changes were applied (as compared to #2), so the png files there are still valid.
If you agree, let me know if you want me to apply it to the dev version (alternatively go ahead to get it applied).
With that I think I'm finished with this issue, unless anybody has remaining requests. If you agree, please go ahead and mark this issue as fixed.
Comment #7
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedComment #8
metzlerd CreditAttribution: metzlerd commentedThis looks good to me.
Comment #10
Pierre.Vriens CreditAttribution: Pierre.Vriens commentedJust committed the proposed changes ... and changing this issue to fixed now ... With that, #2354565: Forena 7.x-4.0 Release has nothing but green child pages! So David, what's next?