[MAGNOLIA-1614] Refactor magnolia Context class hierarchy Created: 02/Jul/07 Updated: 23/Jan/13 Resolved: 06/Nov/07 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 3.1 M1 |
| Fix Version/s: | 3.5 |
| Type: | Task | Priority: | Major |
| Reporter: | Sameer Charles | Assignee: | Philipp Bärfuss |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | Not Specified | Time Spent: | Not Specified |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Attachments: |
|
||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||
| Sub-Tasks: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Description |
|
To avoid issues like there can be various patterns we can use here, I will post more when we start work on it. |
| Comments |
| Comment by Sameer Charles [ 27/Jul/07 ] |
|
To start with I will add login/logout to context this will give us much better control to use these contexts in |
| Comment by Sameer Charles [ 30/Jul/07 ] |
|
needs much more work than anticipated |
| Comment by Philipp Bracher [ 31/Jul/07 ] |
|
The system context should not use ContentRepository to get hierarchy manager instances. It should keep its own instances. In fact ContentRepository should not keep any instances at all. This allows to reinitialize the system context. |
| Comment by Philipp Bracher [ 25/Oct/07 ] |
|
I attached a diagram explaining the changes we do:
|
| Comment by Philipp Bracher [ 25/Oct/07 ] |
|
Attached a diagram showing how the strategies could look like. Broaching the usage of a ContextFactory |
| Comment by Philipp Bracher [ 25/Oct/07 ] |
|
Attached a sequence diagram which should illustrate the roles of ContextFilter, LoginFilter, LogoutFilter and the context methods login(), logout() and release() |
| Comment by Philipp Bracher [ 25/Oct/07 ] |
|
sorry for not rotating the images before uploading |
| Comment by Philipp Bracher [ 26/Oct/07 ] |
|
All the existing interfaces (Context, WebContext) and MgnlContext should not change at all. |
| Comment by ashapochka [ 29/Oct/07 ] |
|
Context Refactoring |
| Comment by ashapochka [ 29/Oct/07 ] |
|
Strategies in Context |
| Comment by ashapochka [ 29/Oct/07 ] |
|
WebContext Creation |
| Comment by ashapochka [ 06/Nov/07 ] |
|
The strategies are created, AnonymousContext is not used in ContextFilter anymore. A single web context is created and then can be logged in / logged out. More testing concerning the correct set up of permissions and jcr for an anonymous is needed. |