[
https://issues.apache.org/jira/browse/QPID-7197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16400253#comment-16400253
]
Keith Wall commented on QPID-7197:
----------------------------------
The change on the whole looks good to me. My comments are:
* The exception message needs to be in language understood by the user. The
term "Configured Object" has no currency. The message should be in expressed in
terms of the objects' names and the category class names. I am relaxed about
the camel casing.
* Even though the current model doesn't need it (with the elimination of
bindings as children), the code should still check that children of the object
being deleted aren't referenced outside the subtree being deleted. I suggest
extending the test hierarchy to give us a convenient test case.
> [Broker-J] Prevent deletion of objects that are in use
> ------------------------------------------------------
>
> Key: QPID-7197
> URL: https://issues.apache.org/jira/browse/QPID-7197
> Project: Qpid
> Issue Type: Improvement
> Components: Broker-J
> Reporter: Keith Wall
> Assignee: Keith Wall
> Priority: Major
> Fix For: qpid-java-broker-7.1.0
>
> Attachments:
> 0001-QPID-7197-Broker-J-Generalize-a-validation-of-refere.patch
>
>
> Enhance the facilities of the CO framework to prevent the deletion of the
> objects that are in use elsewhere in the model. Specifically this means
> searching for other COs that have an attribute that refers to the object that
> is about to be deleted,.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
