Fix handling of unset AppConfig model properties.

Fix handling of unset AppConfig model properties.

BUG=None
R=agable@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219790

[agable, 2013-08-27 14:35:15]

https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py
File app_config.py (right):

https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py#newco...
app_config.py:64: (field, field, cgi.escape(getattr(app_config, field) or '')))
can simply use getattr(app_config, field, '') instead.

[Mattias Nissler (ping if slow), 2013-08-27 14:40:09]

https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py
File app_config.py (right):

https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py#newco...
app_config.py:64: (field, field, cgi.escape(getattr(app_config, field) or '')))
On 2013/08/27 14:35:15, Aaron Gable wrote:
> can simply use getattr(app_config, field, '') instead.

That's what I had before. It doesn't work in this case the attribute is present
but its value is None, so getattr(app_config, field, '') will return None.

[agable, 2013-08-27 14:47:02]

On 2013/08/27 14:40:09, Mattias Nissler wrote:
> https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py
> File app_config.py (right):
> 
>
https://chromiumcodereview-hr.appspot.com/23602002/diff/1/app_config.py#newco...
> app_config.py:64: (field, field, cgi.escape(getattr(app_config, field) or
'')))
> On 2013/08/27 14:35:15, Aaron Gable wrote:
> > can simply use getattr(app_config, field, '') instead.
> 
> That's what I had before. It doesn't work in this case the attribute is
present
> but its value is None, so getattr(app_config, field, '') will return None.

Ah, sad. LGTM.

[Mattias Nissler (ping if slow), 2013-08-27 14:51:19]

Committed patchset #1 manually as r219790 (presubmit successful).