This module provides the exact functionality that I need for a current project. I plan to fix a number of the bugs, bring the code a bit closer to Drupal standards, and add a few features.
Given that I'll be making multiple modifications, I thought I'd offer to co-maintain the module for the near future. If not, I'll start opening multiple issues and posting patches, which will be more time consuming.
Let me know if you're interested. Current items I'm working on:
- fixing nodeownership_node_view() to display link as part of node links array
- moving configuration page under admin/config, in compliance with Drupal standards
- possibly implementing workflow to walk anonymous users through attempt claim -> register -> claim
- possible views integration to simplify admin interface
- rules integration
I may also be able to knock off a few items from the current todo list.
Let me know if you're interested.
Comments
Comment #1
grasmash commentedI've completed the following items thus far:
I'll probably do a bit more clean up, including breaking the UI into a separate, views-dependent module, and breaking the page callbacks out into include files for better performance.
Let me know if you're interested in these changes. If so, I can push to a 7.x-2.x branch to keep things separate, if you'd like.
Comment #2
grasmash commentedI've uploaded the changes to a sandbox:
http://drupal.org/sandbox/madmatter23/1508694
Haven't tested the hook_update_n() implementation on a clean install yet, but it should work. There a new 'id' id row in the db that acts as a primary auto_inc key.
Comment #3
vyasamit2007 commented@madmatter23 : I've added you as a co-maintainer for this module. You can write to the VCS now. Please upload your changes in the dev branch.
Thanks for resolving these bugs/features.
Thanks!!
Amit
Comment #4
grasmash commentedGreat, thanks.
I've pushed the changes to dev. I plan to test the upgrade from 7.x-1.1 to 7.x-1.x-dev later today to smooth out any upgrade path issues. The hook_update_n() is currently untested. Otherwise, everything seems very functional.
Comment #5
grasmash commentedHm. Even after pushing to dev, the development snapshot on the project page is still dated at March 17. Are new snapshots currently disabled for this project?