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
StatusFileSize
new674 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
StatusFileSize
new610 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

StatusFileSize
new610 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.