CGFeedback: Undefined index in class.comment.php

[Andrew Prior]

My error log has been growing over 1mb per day, which I suspect is not normal. I have found a couple of the causes and removed them. I have three remaining issues. The first concerns CG Feedback and pops up 1-5 errors in succession every few minutes

The Error: [25-Oct-2017 08:54:56 Australia/Adelaide] PHP Notice: Undefined index: extra in /home/mansweb/public_html/modules/CGFeedback/lib/class.comment.php on line 18

Lines 13-21 of class.comment.php

Code: Select all

    
  else if( is_array($this->_data) && is_array($this->_data['extra']) && isset($this->_data['extra'][$key]) ) {
    

I have had trouble getting the module to work, but all now seems dandy. However the sheer number of errors is a bit concerning.

Any clues? Can I just ignore this?

Andrew

----------------------------------------------
Cms Version: 2.2.3.1
Installed Modules:
AceEditor2: 1.05
AdminSearch: 1.0.4
Album: 1.11
CGActionBar: 1.1.2
CGExtensions: 1.57.1
CGFeedback: 2.1
CGJobMgr: 1.3.6
CGSimpleSmarty: 2.1.7
CMSContentManager: 1.1.6
CMSMailer: 6.2.14
Captcha: 0.5.5
CmsJobManager: 0.1.1
DesignManager: 1.1.3
FileManager: 1.6.4
FilePicker: 1.0.1
JQueryTools: 1.4.0.3
MenuManager: 1.50.3
MicroTiny: 2.2.1
ModuleManager: 2.1.2
NMS: 2.12.2
Navigator: 1.0.8
News: 2.51.3
Printing: 1.1.2
Search: 1.51.2
Showtime2: 3.5.4
SiteMapMadeSimple: 1.2.8
ThemeManager: 1.1.8
nuSOAP: 1.0.2

Config Information:
php_memory_limit:
max_upload_size: 16000000
url_rewriting: mod_rewrite
page_extension: .html
query_var: page
auto_alias_content: true
locale:
set_names: false
timezone: Australia/Adelaide
permissive_smarty: true

Php Information:
phpversion: 5.6.31
md5_function: On (True)
json_function: On (True)
gd_version: 2
tempnam_function: On (True)
magic_quotes_runtime: Off (False)
E_ALL: 32767
E_STRICT: 2048
E_DEPRECATED: 8192
test_file_timedifference: No time difference found
test_db_timedifference: No time difference found
create_dir_and_file: 1
memory_limit: 128M
max_execution_time: 120
register_globals: Off (False)
output_buffering: 4096
disable_functions:
open_basedir:
test_remote_url: Success
file_uploads: On (True)
post_max_size: 32M
upload_max_filesize: 16M
session_save_path: /tmp (0700)
session_use_cookies: On (True)
xml_function: On (True)
xmlreader_class: On (True)
check_ini_set: On (True)
curl: On

Performance Information:
allow_browser_cache: Off (False)
browser_cache_expiry: 30
php_opcache: On (True)
smarty_cache: Off (False)
smarty_compilecheck: Off (False)
auto_clear_cache_age: On (True)
Server Information:
Server Software: Apache
Server Api: cgi-fcgi
Server Os: Linux 3.10.0-714.10.2.lve1.4.61.el7.x86_64 On x86_64
Server Db Type: MySQL (mysqli)
Server Db Version: 5.6.37
Server Db Grants: Found a "GRANT ALL" statement that appears to be suitable

Permission Information:
tmp: /home/mansweb/public_html/tmp (0755)
tmp_cache: /home/mansweb/public_html/tmp/cache (0755)
templates_c: /home/mansweb/public_html/tmp/templates_c (0755)
modules: /home/mansweb/public_html/modules (0755)
uploads: /home/mansweb/public_html/uploads (0775)
File Creation Mask (umask): /home/mansweb/public_html/tmp/cache (0755)
config_file: 0444
----------------------------------------------

[velden]

Note that those are not errors but notices.
For production environments you better lower the log level (or don't worry about a few MBs of logs).

In some cases a notice can be a hint to an error that occurs later on in the script. In this specific case it will be fine.

[Andrew Prior]

Thank you Velden.
This is probably a dumb question, but what triggers the notice?
I ask this because I have finally tracked down and cleared another error in the logs which was very clearly triggered by the loading of a page. That is, If I, or someone else went to a page, the error was triggered.

But in the case of these notices from CGFeedback, there is no correspondence to the visitor rate. (I only wish there was!! 3000+ an hour would be pretty nice!!) Andrew

[Rolf]

There is a minor sequence issue in the line you posted, better add a bug report for the developer in the Forge.

grtz. Rolf