search

Arachnid / bloggart

A blog application for App Engine
http://bloggart-demo.appspot.com/
287 stars 78 forks source link

Set bug : def _regenerate_sitemap(): #26

Closed sylvainvivien closed 15 years ago

sylvainvivien commented 15 years ago

I've finished to update bloggartornado with the last changes but I've an error here :

set('/sitemap.xml', rendered, 'application/xml', False) TypeError: set expected at most 1 arguments, got 4

To fix my version : import static and static.set Maybe you have the same issue ?

def _regenerate_sitemap(): import static paths = _get_all_paths() rendered = render_template('sitemap.xml', {'paths': paths}) static.set('/sitemap.xml', rendered, 'application/xml', False)

sylvainvivien commented 15 years ago

Another thing, if there is an error with the set 'keyword' (i.e. : set / frozenset), maybe the set function in static should be renamed : set_static or something like that.

Arachnid commented 15 years ago

Quite right - thanks. The name shouldn't be too much of a problem because it's almost universally called from outside the static module, where it's fully qualified.

Arachnid commented 15 years ago

Fixed missing import. Closed by d920aa8eb53d1904c1386325af9c801772519112.