Last night, I upgraded a PostNuke installation to Thursday's Drupal-CVS. In working to upgrade some users to a new permission role I created, I found that the user account search function does not work. Strangely, when I visit Administration » user management » search accounts, there is a 0 already present in the search field and users with a zero in their username are already listed as matching users. Wildcards do not work, nor will the search return the proper query response even if I insert an exisiting user id. No matter what I enter, it merely redisplays the search field with a zero in it and the user accounts that contain zeros.

Comments

moshe weitzman’s picture

I think my last patch broke this. I can make it work as before, but I don't particularly like that implementation. Any reason why search.module has to handle searching for users. I'd prefer for user.module to handle this. Anyone object?

jb605’s picture

I don't know the inner much, so I don't know whether user.module or search.module should handle this. But I think we should provide more flexible search if possible, i.e., search be within 10 days etc. For that, probably most search should go into search.module, or maybe user.module can then export a interface to search.module. I am not sure if that's possible, because then if we add other new node type, which can be used as search creteria, then they can all export a similar hook to search. Then search probably should be come a standard part instead of a module.

Thank you for reading this.

Dries’s picture

I don't object, Moshe, as long it is clean and simple. Are you going to fix this bug?

Dries’s picture

Assigned: Unassigned » Dries

Just fixed this bug. Marking it as such and assigning it to me instead. (Moshe is very busy this week.)

Anonymous’s picture