Download & Extend
Issues

fix references to nonexistent function drupal_initialize_variables()

Posted by ksenzee on May 14, 2012 at 10:13pm
Project:Drupal core
Version:7.x-dev
Component:documentation
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:needs backport to D7

Issue Summary

We renamed drupal_initialize_variables() to drupal_environment_initialize() in the D7 cycle at #517542: Rename initialization functions for clarity but left a couple of references to it in docblocks.

Login or register to post comments

Comments

#1

Posted by ksenzee on May 14, 2012 at 10:15pm
Status:active» needs review
AttachmentSizeStatusTest resultOperations
1578590-1.env-init-doc.patch1.49 KBIdlePASSED: [[SimpleTest]]: [MySQL] 36,616 pass(es).View details

#2

Posted by System Message on May 14, 2012 at 10:38pm
Status:needs review» needs work

The last submitted patch, 1578590-1.env-init-doc.patch, failed testing.

#3

Posted by jhodgdon on May 16, 2012 at 3:17pm
Status:needs work» needs review

#1: 1578590-1.env-init-doc.patch queued for re-testing.

#4

Posted by jhodgdon on May 16, 2012 at 3:18pm
Status:needs review» reviewed & tested by the community

The patch looks fine to me. Thanks! I'll get it committed (assuming the test bot agrees this time). There's no way this docs patch should have triggered a test failure (must have been a commit glitch yesterday I guess).

#5

Posted by Dries on May 17, 2012 at 1:27am
Version:8.x-dev» 7.x-dev

Great! Committed to 8.x. Thanks.

#6

Posted by jhodgdon on May 17, 2012 at 12:51pm
Status:reviewed & tested by the community» patch (to be ported)

The patch text needs to be adjusted for Drupal 7.

#7

Posted by ksenzee on May 17, 2012 at 11:14pm
Status:patch (to be ported)» needs review
AttachmentSizeStatusTest resultOperations
1578590-7.env-init-doc.patch1.23 KBIdlePASSED: [[SimpleTest]]: [MySQL] 39,125 pass(es).View details

#8

Posted by System Message on May 17, 2012 at 11:53pm
Status:needs review» needs work

The last submitted patch, 1578590-7.env-init-doc.patch, failed testing.

#9

Posted by jhodgdon on May 18, 2012 at 2:36pm

Looks good. I'll hit retest just in case... UpgradePath tests are failing...

#10

Posted by jhodgdon on May 18, 2012 at 2:36pm
Status:needs work» needs review

#7: 1578590-7.env-init-doc.patch queued for re-testing.

#11

Posted by jhodgdon on May 18, 2012 at 2:37pm
Status:needs review» reviewed & tested by the community

pending tests passing...

#12

Posted by System Message on May 18, 2012 at 3:15pm
Status:reviewed & tested by the community» needs work

The last submitted patch, 1578590-7.env-init-doc.patch, failed testing.

#13

Posted by ksenzee on May 18, 2012 at 7:07pm
Status:needs work» needs review

#7: 1578590-7.env-init-doc.patch queued for re-testing.

#14

Posted by jhodgdon on May 21, 2012 at 2:53pm
Status:needs review» fixed

Thanks! Committed to 7.x.

#15

Posted by System Message on June 4, 2012 at 3:01pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.