Uploaded image for project: 'SX Spring Python'
  1. SX Spring Python
  2. SESPRINGPYTHONPY-111

Move InitializingObject from springpython.factory to springpython.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0.M2
    • Component/s: IoC
    • Labels:
      None

      Description

      Need to move
      class InitializingObject(object):
      """This allows definition of a method which is invoked by the container after an object has had all properties set."""
      def after_properties_set(self):
      pass

      from springpython.factory over to springpython.context.

        Activity

        Hide
        dsuch Dariusz Suchojad added a comment -

        In that case, I'll do the same with springpython.factory.DisposableObject.

        Show
        dsuch Dariusz Suchojad added a comment - In that case, I'll do the same with springpython.factory.DisposableObject.
        Hide
        gregturn Greg Turnquist added a comment -

        Figured out reason this code was NOT in context. springpython.context dependends on PyroProxyFactory, in order to mark as not an option for certain context behaviors. However, having springpython.remoting.pyro import springpython.context generates a circular import. Only solution was to duck type one side, to remove the import. Decided that springpython.remoting.pyro would be the side NOT importing springpython.context.

        Show
        gregturn Greg Turnquist added a comment - Figured out reason this code was NOT in context. springpython.context dependends on PyroProxyFactory, in order to mark as not an option for certain context behaviors. However, having springpython.remoting.pyro import springpython.context generates a circular import. Only solution was to duck type one side, to remove the import. Decided that springpython.remoting.pyro would be the side NOT importing springpython.context.

          People

          • Assignee:
            gregturn Greg Turnquist
            Reporter:
            gregturn Greg Turnquist
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: