#2 Use CMFCore if FileSystemSite is not available

closed-accepted
nobody
None
5
2006-09-22
2006-09-18
Florian Schulze
No

We use CMFCore because FileSystemSite caused us some issues. The
attached patch adds conditional import of CMFCore if FileSystemSite is
not found.

Discussion

  •  
    Attachments
  • Kevin Campbell
    Kevin Campbell
    2006-09-19

    Logged In: YES
    user_id=1241761

    Accepted pending contributers agreement. Can be added to the
    2.1 branch for 2.1.1

     
  • Kevin Campbell
    Kevin Campbell
    2006-09-19

    • status: open --> open-accepted
     
  • Kevin Campbell
    Kevin Campbell
    2006-09-22

    • status: open-accepted --> closed-accepted
     
  • Kevin Campbell
    Kevin Campbell
    2006-09-22

    Logged In: YES
    user_id=1241761

    Applied to the 2.1 branch now, and updated the doc files.
    Thanks for the submission.

     
  • Logged In: YES
    user_id=18721

    You missed one:
    Index: migrations/v2_0/__init__.py
    ===============================================
    ====================
    --- migrations/v2_0/__init__.py (revision 3628)
    +++ migrations/v2_0/__init__.py (working copy)
    @@ -20,7 +20,10 @@
    from psycopg import Binary
    except ImportError:
    pass
    -from Products.FileSystemSite.DirectoryView import
    manage_addDirectoryView
    +try:
    + from Products.FileSystemSite.DirectoryView import
    manage_addDirectoryView
    +except ImportError:
    + from Products.CMFCore.DirectoryView import manage_addDirectoryView

    # Python library modules.
    import os