Feature #35

Refactor T3P_Core_AOPFramework: Extract build methods

Added by Robert Lemke 590 days ago. Updated 266 days ago.

Status:Closed Start:
Priority:Should have Due date:
Assigned to:Robert Lemke % Done:

0%
Category:AOP
Target version:0.1.0

Description

The T3P_Core_AOPFramework got a bit too large by the time and it makes sense extracting the whole proxy code building methods into its own class. While doing that, the code snippets should also be externalized and put into template files where it makes sense.

History

2007-03-01 15:24 - Robert Lemke

(In r159) * Core: Removed parsing support for "lifecycleDestructionMethod"
  • Reflection: Added doc tag parsing support for properties (created T3P_Reflection_Property)
  • Core: Implemented support for AOP Introduction declarations. Works, but needs a bit of a cleanup in T3P_Core_AOPFramework. Refs #35 and fixes #20.

2007-03-01 19:38 - Robert Lemke

(In r160) * Core: Refactored the T3P_Core_AOPFramework: The builder functions are now in their own class and have been consolidated. Still needs some cleanup in the method interceptor code generator. Addresses #35.

2007-03-04 18:06 - Robert Lemke

  • Status changed from New to Closed
  • Sprint set to fixed

Okay, I close this ticket for now - it's enough refactoring for the moment. If we feel like putting the code snippets into a template file, we can do so later on.

2007-03-05 20:24 - Robert Lemke

(In r167) Refactored the T3P_Core_AOPProxyClassBuilder:

  • Core: The AOPProxyClassBuilder smelled like Duplicated Code (76) and therefore I decided to split it up using the Template Method (325) pattern. Addresses #35.
  • Core: Added tests and fixtures to assert that introducing multiple interfaces does work (it didn't work as my tests showed soon). Now everything seems to be fine and that fixes #37.

2007-03-06 00:07 - Karsten Dambekalns

(In r169) Fixes failures in T3P_phpCRJackrabbit_CheckPermissionTest caused by incorrectly set up account permissions:
  • repository.xml now does no longer provide a default user for logins without credentials
  • The unit test base class uses the configured "anonymous" user for read-only session access
    This fixes #35

Additionally T3P_phpCRJackrabbit_CheckPermissionTest contained some uses of + instead of . for string concatenation, this created invalid input.

2007-03-06 00:11 - Karsten Dambekalns

Replying to [comment:5 karsten]:

(In r169) Fixes failures in T3P_phpCRJackrabbit_CheckPermissionTest caused by incorrectly set up account permissions:

Sorry, wrong ticket id used in commit message. Can this be corrected?

Also available in: Atom PDF