Details
-
Sub-task
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
None
-
None
Description
I think that it's important for the user to know what Slingclipse is doing, especially since it uses some heuristics to detect Sling-based projects.
My first thoughts would be:
- from a UI point of view start with something simple like a text-only console . The subclipse console is a good starting point, we can simply log things like
ADD $SLING_URL $PROJECT/$RESOURCE 200 OK
REMOVE $SLING_URL $PROJECT/$RESOURCE 500 Internal Server Error
- from an implementation point of view we could use something like a command pattern ( SaveResource command, DeleteResourceCommand ) . This would have the benefit of allowing this logging easily + encapsulating the operations for batching, handling of offline operations, redo etc.
Thoughts? I'm willing to look into this, at least for a first iteration.
Attachments
Attachments
Activity
rmuntean
created issue -
rmuntean
made changes -
Field | Original Value | New Value |
---|---|---|
Attachment | SLING-2634-proof-of-design.diff [ 12552130 ] |
Antonio Sanso
made changes -
Assignee | Antonio Sanso [ asanso ] |
rmuntean
made changes -
Attachment | SLING-2634-1.diff [ 12552309 ] |
Antonio Sanso
made changes -
Resolution | Fixed [ 1 ] | |
Status | Open [ 1 ] | Resolved [ 5 ] |
rmuntean
made changes -
Attachment | SLING-2634-2.diff [ 12552629 ] |
Gavin McDonald
made changes -
Workflow | no-reopen-closed,doc-test-required [ 12731548 ] | Copy of no-reopen-closed,doc-test-required [ 12765180 ] |
Gavin McDonald
made changes -
Workflow | Copy of no-reopen-closed,doc-test-required [ 12765180 ] | no-reopen-closed,doc-test-required [ 12767927 ] |
Gavin McDonald
made changes -
Reporter | Robert Munteanu [ rmuntean ] | Robert Munteanu [ rombert ] |
Gavin McDonald
made changes -
Workflow | no-reopen-closed,doc-test-required [ 12767927 ] | re-open possible,doc-test-required [ 12790236 ] |
Gavin McDonald
made changes -
Workflow | re-open possible,doc-test-required [ 12790236 ] | no-reopen-closed,doc-test-required [ 12792825 ] |
Robert Munteanu
made changes -
Component/s | IDE [ 12320908 ] | |
Component/s | Extensions [ 12312240 ] | |
Fix Version/s | Sling Eclipse IDE 1.0.0 [ 12324873 ] |
Stefan Egli
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
rmuntean I am not sure I really caught your subclipse example.
For the Command pattern idea it was actually my first implementation but it did not really combined well with the declarative services (or at least my command pattern implementation )