zend_ready() functions $path argument seems unused.

CommentFileSizeAuthor
#1 zend_path.patch1.75 KBrobloach

Comments

robloach’s picture

robloach
he/him
commented
Status: Active » Needs review
StatusFileSize
new zend_path.patch1.75 KB

The reason $path was being passed in was for the validation of the settings page. Since it's not validating the path anymore, it seems that we can remove the argument. Any thoughts on this?

It'll have to go into the Drupal 6 branch too.

mustafau’s picture

mustafau commented
Status: Needs review » Reviewed & tested by the community

I think this patch is ready to be commited.

robloach’s picture

robloach
he/him
commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks a lot for checking this out! http://drupal.org/cvs?commit=115131

robloach’s picture

robloach
he/him
commented
Status: Patch (to be ported) » Fixed

$path is not there in Drupal 5.

Status: Fixed » Closed (fixed)

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