On line 32, unknown function "db_result". Once I fixed it a new bug appeared in db_query_range. Here is my fix of the bug.

Comments

erikwebb’s picture

Title: Bug in the Drupal 7 version in constraint_delay.inc » constraint_delay.inc still uses D6 database functions
Status: Active » Needs work

Could you submit this as a proper patch file?

kmasood’s picture

Doesn't look like this made it ... will it be part of beta3?

KM

erikwebb’s picture

Version: 7.x-1.0-beta1 » 7.x-1.x-dev

If this can be converted to a proper patch, I'd be glad to review it.

kmasood’s picture

StatusFileSize
new1.07 KB

Thanks @erikwebb. Patch attached based on what @angel.h provided.

KM

BTW, this is the first patch for me, so lemme know if I messed up, I'll gladly resubmit.

erikwebb’s picture

Status: Needs work » Needs review
StatusFileSize
new860 bytes

I reformatted your patch a little bit. Also, when submitting a patch, make sure to mark the issue "needs review" to get the attention of reviewers.

kmasood’s picture

Thanks @erikwebb. Lemme know if you need help testing this out.

erikwebb’s picture

Yes, please. I currently don't have a good setup for testing this out.

matt v.’s picture

Status: Needs review » Reviewed & tested by the community

I had trouble getting the patch to apply for some reason, but it was easy enough to apply by hand. The change fixed the "unknown function" error for me.

erikwebb’s picture

Status: Reviewed & tested by the community » Fixed

Matt V. - What problems did you have? Make sure to use "git apply" instead of the patch command (http://drupal.org/node/707484).

Fixed and committed.

http://drupalcode.org/project/password_policy.git/commit/932de99

Status: Fixed » Closed (fixed)

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