I just upgraded from nodeownership 7.x-1.0 to 7.x-1.x-dev, and I've found a number of errors that seem to step from using Drupal 6 code. It seems like some of the D7 changes in 7.x-1.0 were somehow overwritten or reverted in 7.x-1.x-dev.

Examples:

  • nodeownership_perm() is called rather than nodeownership_permission()
  • nodeownership_admin_settings() calls node_get_types(), which does not exist in D7.
  • nodeownership_nodeapi() is called rather than nodeownership_node_view()
  • DBTNG is not being used

Should I be pulling from another branch to see the most recent code?

Comments

grasmash’s picture

grasmash commented
Issue summary: View changes

fixing hook_permission statement

grasmash’s picture

grasmash commented
Issue summary: View changes

added nodeownership_node_view() example

grasmash’s picture

grasmash commented

strangely, these issues don't seem to exist on the 7.x-1.x branch of the git repository.

grasmash’s picture

grasmash commented
Status: Active » Fixed

This issue should be fixed with the new dev version, pushed out today.

Status: Fixed » Closed (fixed)

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

coloryan’s picture

coloryan commented
Status: Closed (fixed) » Active

Can you confirm the update on March 31? The latest appears to be 2012-Mar-18 and I'm having this issue.

grasmash’s picture

grasmash commented

I can confirm that this has been pushed to the repository, but it doesn't seem that it's being pushed to the nightly dev snapshots. You'll have to get the code directly from the repository.

I've tried contacting the maintainer multiple times about gaining access to post releases, but he has been unresponsive.

gauravkumar87’s picture

gauravkumar87 commented

Hi Madmatter23,

I've given you the 'Administer Releases' permission. My apologies for the delay in response.

Gaurav

grasmash’s picture

grasmash commented
Status: Active » Fixed

Thanks! I've pushed out a 7.x-1.2. I'll keep track of any bugs that may arise from my changes.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

added DBTNG example