(Like all of the patches which I'm currently posting, this is based on #1164578: Port and extend tests so that one needs to be commited first as this one mit not apply cleanly otherwise)

Thanks to drupal_static(), the $reset argument can now be removed from most functions.

I left it only for the load functions, as does core..

CommentFileSizeAuthor
#2 reset2.patch1.88 KBberdir
reset.patch1.77 KBberdir

Comments

simon georges’s picture

simon georges commented
Component: User interface » Code
Status: Needs review » Needs work

Could you update the documentation when you remove the parameter, please ?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Needs review
StatusFileSize
new reset2.patch1.88 KB

Sure, only one of those functions had it documented, though.

simon georges’s picture

simon georges commented

Thanks.

simon georges’s picture

simon georges commented
Status: Needs review » Fixed

Committed. Thanks again!

Status: Fixed » Closed (fixed)

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