We're receiving a lot of warnings in the log from the htmLawed process callback:

Warning: Missing argument 6 for _htmLawed_process(), called in /home/{user}/public_html/sites/all/modules/ckeditor/includes/ckeditor.page.inc on line 241 and defined in _htmLawed_process() (line 32 of /home/{user}/public_html/sites/all/modules/htmLawed/htmLawed.module).

This is referring to the $langcode, $cache, $cache_id defined in _htmLawed_process(), however they don't seem to be referenced in the function body. Should these perhaps have some defaults defined or does this need to be handled by CKEditor?

Comments

alpha2zee’s picture

alpha2zee commented

It seems that this is something to do with CKEditor; see this posting. I looked at the codes of a couple of filter modules and none of them seem to provide a default value for the 4th-6th arguments. See line 73 of link_node.module file for the link node module, or line 76 of pathologic.module for the Pathologic module. But then, may be no one has yet reported the PHP warnings that occur with those modules.

hephaestus’s picture

hephaestus commented

Thanks for the response, I've posted this issue to the CKEditor tracker.

In the mean time, would you be able to suggest some sensible defaults I can patch in as a work around, in order to not have the logs flooded by these warnings?

hephaestus’s picture

hephaestus commented
Status: Active » Closed (works as designed)

This has been confirmed as having been fixed in the current dev release of CKEditor.

hephaestus’s picture

hephaestus commented
Issue summary: View changes

Fixed paths.