Search the web
Sign In
New User? Sign Up
mt-dev · Movable Type Developers
? Already a member? Sign in to Yahoo!

Yahoo! Groups Tips

Did you know...
Want to share photos of your group with the world? Add a group photo to Flickr.

Best of Y! Groups

   Check them out and nominate your group.
Having problems with message search? Fill out this form to ensure your group is one of the first to be migrated to the new message search system.

Messages

  Messages Help
Advanced
MT::Task subclassing not fully supported   Message List  
Reply | Forward Message #1348 of 2112 |
Here is one I just found and reported.

The add_task method in MT::Compat::v3 doesn't handle a MT::Task
subclasses. Line 181 reads like this...

} elsif (ref($task) eq 'MT::Task') {

It should read something like...

} elsif (ref($task) && $task->isa('MT::Task')) {

While debugging this I also noticed that MT::Task breaks encapsulation
in the name method.

$task->{name} || $task->{key};

Calling the key hash directly by-passes an overloaded method like
mine. It's a small thing, but a potential problem under the right
circumstances.

Are there any prizes for finding the most obscure bug? ;)

<tim/>

--
Timothy Appnel
Appnel Solutions
http://appnel.com/

Blogcast / Powerful blogging systems made easy
http://blogcast.net/



Wed Jul 18, 2007 4:51 pm

tappnel
Offline Offline
Send Email Send Email

Forward
Message #1348 of 2112 |
Expand Messages Author Sort by Date

Here is one I just found and reported. The add_task method in MT::Compat::v3 doesn't handle a MT::Task subclasses. Line 181 reads like this... } elsif...
Timothy Appnel
tappnel
Offline Send Email
Jul 18, 2007
4:51 pm
Advanced

Copyright © 2009 Yahoo! Inc. All rights reserved.
Privacy Policy - Terms of Service - Guidelines - Help