Re: Creating custom propertis plugin

Dan Rollo <[email protected]> Thu, 29 Sep 2011 19:35:57 -0400
Newsgroups gmane.comp.java.cruise-control.devel
Message-ID <[email protected]>
I have never actually used the registerCustomProperty(), so I don't 
think I can add much. I wonder if it could be used via a direct call (at 
least for testing), and later any missing "addxxx()" methods could be added?

Sorry, but I don't know the details of the intended use case here. This 
feels like a case where I wish we had a unit test that could show us the 
intent.

Dan (R)



Date: Wed, 03 Aug 2011 22:42:38 +0200
From: "Ing. Daniel TIHELKA Ph.D." <[email protected]>
Subject: Re: [Cruisecontrol-devel] Creating custom propertis plugin
To: [email protected]

Hallo,
I have looked around little bit and I have found the following:

The only place where raw

   PropertiesPlugin#loadProperties(Map<String, String>, boolean)

is called, is:

   PropertiesPlugin ProjectXMLHelper#registerCustomProperty(Map<String, 
String>,
             Element, ResolverHolder, boolean, PluginRegistry)

which is called from:

   CruiseControlConfig.handleCustomRootProperty(Element)

which is then called from:

   CruiseControlConfig.parse(Element)


However, when I debugged it with my plugin configuration (see my 
previous mail), such methods are never called (I have put a breakpoint 
there and debugged the instantiation of  new XMLConfigManager(projectFile)).


So, the question is - what to do now? In fact. I don't know how to 
create custom properties plugin. Simple implementation of 
PropertiesPlugin interface seems not to be enough.

Do you have an idea what needs to be changed/adjusted to get it work 
correctly?

Thank you very much for your help.
Best regards,
Dan T.


------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1