Get this error on /admin/settings/contentanalysis

Notice: Undefined variable: analysis in contentanalysis_get_analyzer_form_element() (line 380 of /sites/all/modules/contentanalysis/contentanalysis.module).
Notice: Undefined variable: a in contentanalysis_admin_settings() (line 68 of /sites/all/modules/contentanalysis/contentanalysis.admin.inc).

CommentFileSizeAuthor
#9 contentanalysis-1242966-6.patch216 bytesRefineo
#5 contentanalysis.patch3.88 KBbgilhome
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wizzardo’s picture

wizzardo CreditAttribution: wizzardo commented

It seems the input should read $params and not $analysis Although I'm getting more problems thereafter
Undefined index: sections in contentanalysis_analysis_form() (line 264 of public_html/home/sites/all/modules/contentanalysis/contentanalysis.module).

Gerto’s picture

Gerto CreditAttribution: Gerto commented

Same here, Drupal 7.8

MGParisi’s picture

MGParisi CreditAttribution: MGParisi commented

Subscribe D7.8

kingfisher64’s picture

kingfisher64 CreditAttribution: kingfisher64 commented

Same issue here.

Subscribe D7.8.

bgilhome’s picture

bgilhome CreditAttribution: bgilhome commented
Status: Active » Needs review
FileSize
contentanalysis.patch3.88 KB

This patch updates the drupal_get_form calls to Drupal 7 practice, as well as fixing a couple of 'Undefined variable' errors. Analysis now works from the Content Analysis page but not from node edit forms.

Refineo’s picture

Refineo CreditAttribution: Refineo commented

After applying the patch to D7.8 + Content Analysis 7.x-1.0-alpha2, I receive 1 notice on node/add/[content-type] and then on frontpage :

Notice: Undefined variable: element in contentanalysis_get_analyzer_form_element() (line 416 of /***/sites/all/modules/contrib/contentanalysis/contentanalysis.module).

There were multiple notice messages before.

Patch: http://drupal.org/files/issues/contentanalysis.patch

jscm’s picture

jscm CreditAttribution: jscm commented

subscribe D6.22

Refineo’s picture

Refineo CreditAttribution: Refineo commented

I manually created and applied the small patch below and the notice from #6 disappears.

Refineo’s picture

Refineo CreditAttribution: Refineo commented
FileSize
contentanalysis-1242966-6.patch216 bytes

Here is the patch, I couldn't add to the previous comment #8.

jscm’s picture

jscm CreditAttribution: jscm commented

@refineo - When will be this patch inside a new official release?

Refineo’s picture

Refineo CreditAttribution: Refineo commented

@TomDude48 should decide as it is his project. The patch should be tested first tough.

astutonet’s picture

astutonet
Location Sao Paulo, SP
CreditAttribution: astutonet commented

The patch in #9 solve my issue with undefined analysis, but I have more two issues with undefined index, here.

Tks.

revasm’s picture

revasm CreditAttribution: revasm commented

Having the same undefined variable and undefined index issues on D7.8. Following and hoping the patch will be incorporated into a new release soon...

TomDude48’s picture

TomDude48 CreditAttribution: TomDude48 commented

refineo, thanks for the patch.

I have a bit of a bad habit of working on bugs I know about without looking at the issue queue. I just did the alpha5 release. Please try that one and let me know if you are having the same problem.