[issue64] [PATCH] Fixed query with empty description/summary

Mauricio Teixeira (netmask) at Labix Tracker tracker at labix.org
Mon Dec 26 05:53:17 PST 2005


Mauricio Teixeira (netmask) <mteixeira at webset.net> added the comment:

Although it is some sort of fix, I don't agree it's the best.

IMHO, the correct behavior is that the channel loader should return an empty
string ("", not None) if there is no description for the package. So we should
check the erroneous behavior on the channel loader.

What kind of channels are you using?

Another approach would be:

data = (info.getSummary() or "")

It would be a smaller change, and would result on the same effect (empty string
when info.getSummary returns None, which is erroneous, as I said before).

I think Niemeyer is the right person to abitrate on that issue.

----------
nosy: +netmask
status: unread -> chatting

_______________________________________
Labix issue tracker <tracker at labix.org>
<http://tracker.labix.org/issue64>
_______________________________________



More information about the Smart mailing list