It sets the type to "menu", and instead it should be "node".

dblog showing wrong type

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grendzy’s picture

naxoc’s picture

Status: Needs review » Reviewed & tested by the community

Yes. This looks good to me.

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)
Issue tags: +Needs backport to D7

Yep. Committed/pushed to 8.x.

tim.plunkett’s picture

Status: Patch (to be ported) » Needs review
FileSize
674 bytes

Rerolled.

Devin Carlson’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #4 applied cleanly and successfully changed the type from "menu" to "node".

David_Rothstein’s picture

Version: 7.x-dev » 6.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)
Issue tags: +Needs backport to D6

Thanks! Committed to 7.x: http://drupalcode.org/project/drupal.git/commit/57b8f47

Seems like this could be backported to Drupal 6 also.

Albert Volkman’s picture

Status: Patch (to be ported) » Needs review
FileSize
610 bytes

Backported.

thiagomp’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #7 worked fine for D6.

star-szr’s picture

Assigned: grendzy » Unassigned
Issue tags: -Novice

Looks like this just needs a commit, removing Novice tag.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 7: drupal-1573082-7.patch, failed testing.

tadityar’s picture

Issue summary: View changes
Status: Needs work » Needs review

Re-rolled, maybe it'll pass

tadityar’s picture

FileSize
610 bytes

forgot the patch..

Albert Volkman’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

Status: Reviewed & tested by the community » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.