Is anyone working on this?

CommentFileSizeAuthor
#1 path_blacklist.patch4.52 KBcsko

Comments

csko’s picture

csko commented
Assigned: Unassigned » csko
Status: Active » Reviewed & tested by the community
StatusFileSize
new path_blacklist.patch4.52 KB

Hi, here is my 6.x patch for this module. Works for me, test it.

mitchell’s picture

mitchell commented

It works! path_blacklist.info was rejected for me.

Also, I'm not sure if this is a bug to be filed separately, but I first added "path*", and it didn't block a node with "path". Then I changed it to "path", and it worked.

Thanks csko!

csko’s picture

csko commented

I think I've made a mistake diffing the .info file with the CVS repo, the CVS system adds some extra info, I'll check it out. I might post an other patch later on.

csko’s picture

csko commented
Status: Reviewed & tested by the community » Needs review
crookednumber’s picture

crookednumber commented

Thanks to both of you! I was in the process of making a few tiny changes to v.5, then I'll create the D6 version based on that -- probably this weekend.

@Mitchell: yes, please file this as a separate issue.

mitchell’s picture

mitchell commented

@CrookedNumber: if you need any help testing your patches, I'd be happy to. Looking forward to the new ver.

crookednumber’s picture

crookednumber commented
Status: Fixed » Needs review

Drupal 6 version released. Dev snapshot is up. Official release should be ready later this weekend.

http://drupal.org/project/path_blacklist

Thanks mitchell for the nudge. And csko for the patch.

Most notable change: D6 now has a drupal_match_path function, which saved me a few lines of code.

crookednumber’s picture

crookednumber commented
Status: Needs review » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Closed (fixed)

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