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

dblog showing wrong type

CommentFileSizeAuthor
#12 drupal-1573082-12.patch610 bytestadityar
#7 drupal-1573082-7.patch610 bytesalbert volkman
#4 drupal-1573082-4.patch674 bytestim.plunkett
watchdog_node.png40.45 KBgrendzy
watchdog_node.patch701 bytesgrendzy

Comments

grendzy’s picture

grendzy commented
naxoc’s picture

naxoc commented
Status: Needs review » Reviewed & tested by the community

Yes. This looks good to me.

catch’s picture

catch
he/him
Primary language English
commented
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

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new drupal-1573082-4.patch674 bytes

Rerolled.

devin carlson’s picture

devin carlson commented
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

David_Rothstein commented
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

albert volkman commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new drupal-1573082-7.patch610 bytes

Backported.

thiagomp’s picture

thiagomp commented
Status: Needs review » Reviewed & tested by the community

The patch in #7 worked fine for D6.

star-szr’s picture

star-szr
he/him
Primary language English
commented
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

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

Re-rolled, maybe it'll pass

tadityar’s picture

tadityar commented
StatusFileSize
new drupal-1573082-12.patch610 bytes

forgot the patch..

albert volkman’s picture

albert volkman commented
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.