Download & Extend
Issues

watchdog "Deleted content type" uses wrong type

Posted by grendzy on May 12, 2012 at 3:16am
Project:Drupal core
Version:6.x-dev
Component:node system
Category:bug report
Priority:minor
Assigned:Unassigned
Status:reviewed & tested by the community
Issue tags:needs backport to D6, needs backport to D7, Quick fix

Issue Summary

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

dblog showing wrong type

AttachmentSizeStatusTest resultOperations
watchdog_node.patch701 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 36,580 pass(es).View details | Re-test
watchdog_node.png40.45 KBIgnoredNoneNone
Login or register to post comments

Comments

#1

Posted by grendzy on May 12, 2012 at 3:18am

#2

Posted by naxoc on May 12, 2012 at 2:21pm
Status:needs review» reviewed & tested by the community

Yes. This looks good to me.

#3

Posted by catch on May 15, 2012 at 12:50pm
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.

#4

Posted by tim.plunkett on May 15, 2012 at 1:54pm
Status:patch (to be ported)» needs review

Rerolled.

AttachmentSizeStatusTest resultOperations
drupal-1573082-4.patch674 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 39,104 pass(es).View details | Re-test

#5

Posted by Devin Carlson on May 16, 2012 at 7:51pm
Status:needs review» reviewed & tested by the community

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

#6

Posted by David_Rothstein on May 25, 2012 at 4:25am
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.

#7

Posted by Albert Volkman on June 8, 2012 at 7:46pm
Status:patch (to be ported)» needs review

Backported.

AttachmentSizeStatusTest resultOperations
drupal-1573082-7.patch610 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 190 pass(es).View details | Re-test

#8

Posted by thiagomp on July 3, 2012 at 2:50pm
Status:needs review» reviewed & tested by the community

The patch in #7 worked fine for D6.

#9

Posted by Cottser on January 19, 2013 at 9:15pm
Assigned to:grendzy» Anonymous
Issue tags:-Novice

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

nobody click here