Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [ice-dev] [ice-build] [eclipse/ice] 40a48d: Updated ICEItemDeleteParticipant to utilize new Fo...

Nicely done Alex! That FormTextContentDescriber is a good way to get around that problem.

Jay

On Tue, Oct 20, 2015 at 4:50 PM, GitHub <noreply@xxxxxxxxxx> wrote:
  Branch: refs/heads/jay/projectSupport
  Home:   https://github.com/eclipse/ice
  Commit: 40a48d5ee0e1dc4e387f9ac5a5c58706f5d8bf1e
      https://github.com/eclipse/ice/commit/40a48d5ee0e1dc4e387f9ac5a5c58706f5d8bf1e
  Author: amccaskey <mccaskeyaj@xxxxxxxx>
  Date:   2015-10-20 (Tue, 20 Oct 2015)

  Changed paths:
    M org.eclipse.ice.client/src/org/eclipse/ice/client/common/ICEItemDeleteParticipant.java
    M org.eclipse.ice.datastructures/META-INF/MANIFEST.MF
    A org.eclipse.ice.datastructures/src/org/eclipse/ice/datastructures/form/FormTextContentDescriber.java
    M org.eclipse.ice.persistence.xml.test/src/org/eclipse/ice/persistence/xml/test/XMLFormContentDescriberTester.java
    M org.eclipse.ice.persistence.xml/META-INF/MANIFEST.MF
    M org.eclipse.ice.persistence.xml/src/org/eclipse/ice/persistence/xml/XMLFormContentDescriber.java

  Log Message:
  -----------
  Updated ICEItemDeleteParticipant to utilize new FormTextContentDescriber

Created an extension of the ITextContentDescriber for ICE content
describing that provides a method to get Item specific details, such as
the Item ID. Updated XMLFormContentDescriber to realize this interface,
updated its test to check getItemID.

Updated ICEItemDeleteParticipant to use the available
FormTextContentDescriber from the extension registry.

Signed-off-by: amccaskey <mccaskeyaj@xxxxxxxx>



_______________________________________________
ice-build mailing list
ice-build@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/ice-build




--
Jay Jay Billings
Oak Ridge National Laboratory
Twitter Handle: @jayjaybillings

Back to the top