Hi there. I installed the D7 version of the module to see if it was usable for my site but immediately when I tried accessing the Filter Formats and when I tried to create a new content I got these warnings, and no tips for the available shortcodes appeared (as per screenshot):

Notice: Undefined variable: args in _shortcode_filter_tips() (line 76 of /var/www/site/sites/all/modules/contrib/shortcode/shortcode.module).
Warning: call_user_func_array() expects parameter 2 to be array, null given in _shortcode_filter_tips() (line 76 of /var/www/site/sites/all/modules/contrib/shortcode/shortcode.module).

I dived into the code and found the function and the line reported in the warnings and $args is indeed not being defined anywhere. Changing '$args' to 'array($format, $long)' seems to solve the issue and the tips start appearing on the wysiwyg when creating content. As I might need this module for a D7 project I 'll continue testing and report my findings here.

Thanks a lot for your time and the module.
Pavlos

CommentFileSizeAuthor
shortcode_no_tips.png14.93 KBpavlosdan

Comments

denes.szabo’s picture

Assigned: Unassigned » denes.szabo
Status: Active » Fixed

Committed a fix into the d7-2.x-dev version. Pls. test it!

pavlosdan’s picture

Hi Denes will do. Thanks a lot.

denes.szabo’s picture

Status: Fixed » Needs review

status change only.

pavlosdan’s picture

Status: Needs review » Reviewed & tested by the community

Notices & warnings gone. Thanks

denes.szabo’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Thx. closing this.

(I checked other shortcode modules, there are lot of problems. I will fix all of them.)

pavlosdan’s picture

Cool. Was just about to submit an issue about the wysiwyg integration not working (the list of enabled shortcodes to choose from does not appear in the shortcode box). But you seem on top of it :)

Let me know if you need any help or when you commit something that needs testing. More than happy to help create a stable release for this.

pavlosdan’s picture

Issue summary: View changes

updating array() to array($format, $long)