search

celiao / tmdbsimple

A wrapper for The Movie Database API v3.
GNU General Public License v3.0
583 stars 121 forks source link

Add staticmethod decorators for methods in TMDB class #1

Closed qazwsxpawel closed 10 years ago

qazwsxpawel commented 10 years ago

Without the staticmethod decorators you would get e.g. TypeError: unbound method _request() must be called with TMDB instance as first argument (got str instance instead) and a similar TypeError for _set_attrs_to_values. This was also discussed in StackOverflow thread: http://stackoverflow.com/questions/20550738/typeerror-unbound-method-method-name-must-be-called-with-class-name-instanc.

celiao commented 10 years ago

@qazwsxpawel - Your suggestion to add @staticmethod decorators has been incorporated. Thanks for the catch!

qazwsxpawel commented 10 years ago

Awesome. Thanks for your work!