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/ <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.