CloudFilesStorage.get\_file() should not use "self" when
referencing StorageObjectWrapper.

.. figure:: http://i.imgur.com/ewXe7.png
   :align: center
   :alt: 
   

See that dash after the arrow? I think I remember how Christopher
fixed this in the last bug.

pre word¶
=========

While reviewing the late changes, I found some things, I would like
to be considered.

edit\_media() references attachments.¶
--------------------------------------

``edit_media()`` in ``edit/view.py`` references some attachment
stuff. Is that still valid?

delete\_media\_files() might use itervalues¶
--------------------------------------------

::

    - for handle, listpath in media['media_files'].items():
    + for listpath in media['media_files'].itervalues():

import export code: Use the init/ code¶
---------------------------------------

::

    - setup_global_and_app_config(args.conf_file)
    + global_config, app_config = setup_global_and_app_config(args.conf_file)
    ...
    - config, validation_result = read_mediagoblin_config(args.conf_file)
    - connection, db = setup_connection_and_db_from_config(
    -   config['mediagoblin'], use_pymongo=True)
    + connection, db = setup_connection_and_db_from_config(app_config, use_pymongo=True)

final word¶
===========

These are UNTESTED suggestions to consider. They might break
everything, be completely wrong, or just useful. Take your one and
comment on it. Or create branch and add it here for merging.