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.
The inline 'Analyse content' was prefixed with a bad base_path() call. Even without that it generates no content. It's being bound to contentanalysis_inline_analysis() but this is generating a Javascript error 'Uncaught ReferenceError: contentanalysis_inline_analysis is not defined '
contentanalysis/contentanalysis.js is loaded on the page.
Comment | File | Size | Author |
---|---|---|---|
#7 | 1757644-inline-report.patch | 2.71 KB | SocialNicheGuru |
#4 | contentanalysis-fix_inline_button-1757644-3.patch | 4.28 KB | larryboy360 |
Comments
Comment #1
jonathan_hunt CreditAttribution: jonathan_hunt commentedPrefixing contentanalysis_inline_analysis() works, but now all I get is the "Analyzing..." message for a second and then no further output.
Comment #2
jonathan_hunt CreditAttribution: jonathan_hunt commented$data['body']['output'] is empty in the analyzer example, so no content is appended to the textarea. Changing the js to inject content from data.main['output'] does append some markup but it's not yet usable.
Is there a working example of inline content analysis somewhere?
Comment #3
larryboy360 CreditAttribution: larryboy360 commentedLooking at the most recent code. It looks like the JS namespace change broke the inline report button on content pages. Changing the onclick call to use the namespace will fix the initial call and adding the contantanalysis object for Drupal.settings will fix the other issue. In addition, the last round of code clean-up around 7/17/2013 introduced a bug around parsing straight text by change the logic flow when adding isset commands on the $where and $save variables in the module. I will upload a patch for the suggested changes.
Comment #4
larryboy360 CreditAttribution: larryboy360 commentedHere's a suggested patch for the issues noted in comment #3.
Comment #5
larryboy360 CreditAttribution: larryboy360 commentedNot to be rude or impatient, but would someone be willing to look over the patch I submitted? Thanks.
Comment #6
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedComment #7
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedtried a re-roll
Comment #8
ron_s CreditAttribution: ron_s commentedDid you test this with 7.x-1.0-beta7? I'm not having any issues when "display both dialog and inline reports" are selected, but seeing major issues if "display inline reports only" is set.
See this issue I just posted: https://www.drupal.org/project/contentanalysis/issues/2954763