Over in #904978: Preserve breadcrumbs on edit pages for Rules the following code was swapped over:

-define('RULES_SCHEDULER_PATH', RULES_UI_PATH . '/schedule');
+define('RULES_SCHEDULER_PATH', 'admin/config/workflow/rules/schedule');

This code that hasn't changed is still causing issues in 6.x of rules_scheduler when rules_admin is not enabled.

Notice: Use of undefined constant RULES_ADMIN_SET_PATH - assumed 'RULES_ADMIN_SET_PATH' in include_once() (line 11 of /mnt/www/html/sites/all/modules/rules/rules_scheduler/rules_scheduler.module).

This patch backports the very basic alteration to 6.x to ensure similar functionality.

CommentFileSizeAuthor
#3 2204527-2-rules_undefined_constant_notice.patch1.08 KBadammalone
#1 2204527-1-rules_scheduler_undefined_constant_notice.patch736 bytesadammalone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adammalone’s picture

adammalone
Primary language English
Location Sydney
CreditAttribution: adammalone commented
Title: Use of undefined constant RULES_ADMIN_SET_PATH » Use of undefined constant RULES_ADMIN_SET_PATH [patch attached]
Status: Active » Needs review
FileSize
2204527-1-rules_scheduler_undefined_constant_notice.patch736 bytes
howarddraper@gmail.com’s picture

howarddraper@gmail.com CreditAttribution: howarddraper@gmail.com commented

RTBC

adammalone’s picture

adammalone
Primary language English
Location Sydney
CreditAttribution: adammalone commented
Title: Use of undefined constant RULES_ADMIN_SET_PATH [patch attached] » Use of undefined constant RULES_ADMIN_SET_PATH and RULES_ADMIN_PATH [patch attached]
FileSize
2204527-2-rules_undefined_constant_notice.patch1.08 KB

Looking at this again, there is another case of a variable defined in rules_admin (Rules UI) being used in the rest of the rules suite. This is obviously not going to work if the rules admin module is not enabled.

This patch removes the reliance on rules admin of the rules_forms module in extension of the previous patch.

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
CreditAttribution: tobiasb commented
Status: Needs review » Closed (duplicate)

There is already a issue for that. #1103076: Use of undefined constant RULES_ADMIN_SET_PATH and RULES_ADMIN_PATH