On Tue, Apr 1, 2014 at 2:42 PM, Ruchira Wageesha <[email protected]> wrote:

> I think, it better to go with Approach 1. i.e. Then you are independent
> from the XML configuration and relying on Java API.
>
replying on java API is good
+1,

Better we shift other location[1] also to get code consistency.

>
> BTW, rename variable datasourceManager as DataSourceManager. i.e. exactly
> like the class name.
>
> /Ruchira
>
>
> On Tue, Apr 1, 2014 at 2:32 PM, Madhuka Udantha <[email protected]> wrote:
>
>> Hi,
>>
>> Both approaches are valid
>> Approach 2 will much better to used IMO since
>>
>>
>>    - We have followed approach 2 in ES and UES (in our other
>>    products/apps)
>>    - 'carbon' jaggery module have function called 'loadConfig' and it is
>>    written to get carbon server config and 
>> 'conf\datasources\master-datasources.xml'
>>    is valid config file to get through carbon module
>>
>> Thanks
>>
>> [1]
>> https://github.com/wso2/enterprise-store/blob/master/modules/apps/publisher/config/app.js
>>
>>
>> On Tue, Apr 1, 2014 at 12:41 PM, Tanya Madurapperuma <[email protected]>wrote:
>>
>>> Hi all,
>>>
>>> I need to $ subject in a jaggery app and came across the following two
>>> approaches.
>>> What is the best approach to go ahead with? Or is there any other
>>> approach?
>>>
>>> *Approach 1*
>>>
>>> <%
>>>
>>> var log = new Log();
>>> var datasourceManager =
>>> Packages.org.wso2.carbon.ndatasource.core.DataSourceManager;
>>> var mydatasourceManager = new datasourceManager();
>>>
>>> var coll =
>>> mydatasourceManager.getInstance().getDataSourceRepository().getAllDataSources();
>>> var iterator = coll.iterator();
>>>
>>> while (iterator.hasNext()) {
>>>    log.info(iterator.next().getDSMInfo().getName());
>>> }
>>>
>>>
>>> %>
>>>
>>>
>>> *Approach 2*
>>>
>>> <%
>>>
>>> var log = new Log();
>>> var carbon = require('carbon');
>>> var masterds =
>>> carbon.server.loadConfig('datasources/master-datasources.xml');
>>> var ds =
>>> masterds.*::['datasources'].*::['datasource'].*::['name'].text();
>>> log.info(ds);
>>>
>>> %>
>>>
>>>
>>> Thanks,
>>> Tanya
>>>
>>>
>>> --
>>> Tanya Madurapperuma
>>>
>>> Software Engineer,
>>> WSO2 Inc. : wso2.com
>>> Mobile : +94718184439
>>> Blog : http://tanyamadurapperuma.blogspot.com
>>>
>>
>>
>>
>> --
>> *Madhuka* Udantha
>> Senior Software Engineer
>> Development Technologies
>> WSO2 Inc. : http://wso2.com
>>
>> *Mobile*: +94774066336
>> *Blog*: http://madhukaudantha.blogspot.com/
>>
>
>
>
> --
>
> *Ruchira Wageesha**Associate Technical Lead*
> *WSO2 Inc. - lean . enterprise . middleware |  wso2.com <http://wso2.com>*
>
> *email: [email protected] <[email protected]>,   blog:
> ruchirawageesha.blogspot.com <http://ruchirawageesha.blogspot.com>,
> mobile: +94 77 5493444 <%2B94%2077%205493444>*
>



-- 
*Madhuka* Udantha
Senior Software Engineer
Development Technologies
WSO2 Inc. : http://wso2.com

*Mobile*: +94774066336
*Blog*: http://madhukaudantha.blogspot.com/
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to