Field cleanup should be done in accessor methods (e.g. post.title.strip().replace("\n","") )

haystack / feedme-rss

Automatically exported from code.google.com/p/feedme-rss

0 stars 0 forks source link

Field cleanup should be done in accessor methods (e.g. post.title.strip().replace("\n","") ) #110

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago

Let's start a list of fields on model objects that we always clean when
displaying to the user, and make accessor methods that clean them in one
place.  This way, we don't have to do the cleanup in a bunch of different
places in the database

Original issue reported on code.google.com by marcua@gmail.com on 18 Aug 2009 at 9:31

GoogleCodeExporter commented 9 years ago

As the title suggests,

post.title.strip().replace("\n","")

is one such instance.  What are some others?

Original comment by marcua@gmail.com on 18 Aug 2009 at 9:31

GoogleCodeExporter commented 9 years ago

User's name
Post's contents
Post title
Feed title
...

Original comment by esperte...@gmail.com on 18 Aug 2009 at 11:15