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

dblog showing wrong type

Files: 
CommentFileSizeAuthor
#12 drupal-1573082-12.patch610 bytestadityar
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es). View

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
PASSED: [[SimpleTest]]: [MySQL] 39,104 pass(es). View

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
FAILED: [[SimpleTest]]: [MySQL] 190 pass(es), 0 fail(s), and 220 exception(s). View

Backported.

thiagomp’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #7 worked fine for D6.

Cottser’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
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es). View

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.