Hi
i am trying to use the "set all to capital letters" for my title field and a second text field called "code" in Open Atrium 2 distribution and i am getting the bellow error. Before using it in Open Atrium distribution a try it on a basic Drupal 7.41 version and it work correctly so don't know what i am missing.


Error displayed on Open Atrium Dirtibution

Deprecated function: preg_replace(): The /e modifier is deprecated, use preg_replace_callback instead in rules_data_transforms_action_capitalizeall() (line 404 of /Applications/MAMP/htdocs/brainsfd/sites/all/modules/contrib/rules_data_transforms/rules_data_transforms.rules.inc).
Deprecated function: preg_replace(): The /e modifier is deprecated, use preg_replace_callback instead in rules_data_transforms_action_capitalizeall() (line 404 of /Applications/MAMP/htdocs/brainsfd/sites/all/modules/contrib/rules_data_transforms/rules_data_transforms.rules.inc).
CommentFileSizeAuthor
#5 2643616-5.patch1.02 KBtwistor

Comments

delacosta456 created an issue. See original summary.

delacosta456’s picture

delacosta456 commented

hi
can somebody help me for this error please

waverate’s picture

waverate commented

Any update?

delacosta456’s picture

delacosta456 commented

hi waverate...
nobody answered me and the error is still showing

twistor’s picture

twistor commented
Status: Active » Needs review
StatusFileSize
new 2643616-5.patch1.02 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2643616-5.patch, failed testing.

delacosta456’s picture

delacosta456 commented

hi twistor
thanks for the patch...
i will give a try and let you know if worked

waverate’s picture

waverate commented

Patch #5 fixes the error message.

However, I am having no success with either of the Capitalize all words in a string or Make uppercase of the node title.

    0 ms Reacting on event After updating existing content.
    32.472 ms Evaluating conditions of rule Change Case. [edit]
    32.486 ms AND evaluated to TRUE.

    0 ms Rule Change Case fires.
    0.243 ms Evaluating the action uppercase. [edit]
    1.166 ms Added the provided variable uppercase_text of type text [edit]
    1.184 ms Rule Change Case has fired.

    33.704 ms Finished reacting on event After updating existing content.

Any thoughts?

delacosta456’s picture

delacosta456 commented

hi waverate
i haven't tried the patch but can you please try a "before saving new node" and then create a new content(to fire) to see what will happen?

Thanks

waverate’s picture

waverate commented

delacosta456,

I changed the rule to fire on Before saving content of type my_content. The action is:

Capitalize all words in a string
Parameter: Input text: [node:title]
Provides variables: Capitalize all (capitalizeall_text)

I created a title with all lowercase letters.

No error message. Only log message is my_content: added test_content. However, no change to the title.

delacosta456’s picture

delacosta456 commented

hi waverate
ok that may mean that there are still some bug and may be twistor or somedy else can help us..

however , instead of trying it on a titlte fied can you, please try it with another field on new content node to see what will happen please?

thanks

waverate’s picture

waverate commented

delacosta456,

I tried a text field with no difference.

It could be my setup but I would have thought that I was trying something pretty basic.

delacosta456’s picture

delacosta456 commented

ok
I haven't tried it on my own side but will let you know in few hours what is my own result..

by the way i just have an idea that i may try when on it; you can also try it on your side:

- trying the pach with different version of Rules module. (dev version, recommanded version)

delacosta456’s picture

delacosta456 commented

waverate,

I try the patch on several way , the error is gone but te capitalization did not work