Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.In working on the project port to 6.x, with the head version of cvslog installed on an installation using the drupal.org testing profile, I get the following notices at project/cvs/101 where node 101 is the drupal project:
* notice: Undefined index: commit in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1219.
* notice: Undefined index: user in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1223.
* notice: Undefined index: uid in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1227.
* notice: Undefined index: branch in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1236.
* notice: Undefined index: file in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1242.
* notice: Undefined index: rid in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1247.
* notice: Undefined index: message in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1251.
* notice: Undefined index: rss in /Applications/MAMP/htdocs/port/drupal/sites/all/modules/cvslog/cvs.module on line 1282.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 251671_cvslog_notices_2.patch | 6.35 KB | aclight |
Comments
Comment #1
aclight commentedAlso, at project/developers/101, there's this notice:
Comment #2
aclight commentedAttached patch fixes all the messages reported above as well as some other places where we'd probably get messages but I hadn't run across the error messages yet.
Comment #3
dwwResolved conflicts, reviewed, lightly tested, and committed to HEAD. Thanks!