The current configuration for the storage looks like this:

[mediagoblin]
publicstore_class = StorageBackend
publicstore_base_dir = %(here)s/user_dev/media/public
publicstore_base_url = /mgoblin_media/
publicstore_password = secret

Instead I suggest an alternative configuration scheme:

[storage:publicstore]
class = StorageBackend
base_dir = %(here)s/user_dev/media/public
base_url = /mgoblin_media/
password = secret

I implemented this on my dev/storage_config branch.

There is also an other config option with configobj, sub sub sections:

[storage]
  [ [ publicstore ] ]
  base_dir = %(here)s/user_dev/media/public
  base_url = /mgoblin_media/

  [ [ queuestore ] ]
  base_dir = %(here)s/user_dev/media/queue

I don't know, if this one is "better". I would prefer the first one.

I know I was...There are places in redmine mentioning PEP-8 style guidelines (http://www.python.org/dev/peps/pep-0008/), but there is no prominent mention in the contributor docs. This suggestion just came up again in IRC. This issue needs:

• decide where to mention and link to PEP-8
• add it to the contributor documentation

Currently we use .save() to generate the ObjectId, but this is silly, and actually can cause an error when there already exists something with the same slug (and our auto-slug-generation tool might not catch it). We should do ['_id] = ObjectId() instead to explicitly create this.