From: "Abdullah Shaikh" <abdullah.sha...@viithiisys.com>
I got confused. You meant that you will commit the already provided patch ?.
It was really a cursory review and I must I did not thought about generalising.
I was having ContactMechWorker.getCurrentPostalAddress &
ContactMechWorker.getPartyPostalAddresses, in my mind, what I was thinking
was that to make the getPartyPostalAddress service in java by taking the
code out of the ContactMechWorker methods, and calling this service from the
ContactMechWorker class, or should we let ContactMechWorker methods as it is
? and the getPartyPostalAddress service in simple-method as already provided
?
Let me know.
Yes you are right, but be sure to mimic exactly current behaviours.
Thanks
Jacques
On Fri, Sep 18, 2009 at 1:06 PM, Abdullah Shaikh <
abdullah.sha...@viithiisys.com> wrote:
Hi Jacques,
I will provide the patch as discussed.
Just to be sure, what I need to do is :
1) Create a service to get party postaladdress (getPartyPostalAddress)
2) Call this service from the ContactMechWorker.getCurrentPostalAddress &
ContactMechWorker.getPartyPostalAddresses
The patch that I already submitted, won't suffice, for the
ContactMechWorker methods, I guess. I will need to change it to make it work
as per ContactMechWorker methods, so as to get it called from these methods.
Thanks,
Abdullah
On Fri, Sep 18, 2009 at 12:49 PM, Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:
Hi Abdullah,
I had a quick look and it seems good. I will commit I guess.
Thanks (and thank you to Sumit for the link and 1st review)
Jacques
From: "Abdullah Shaikh" <abdullah.sha...@viithiisys.com>
I feel that we should have this service,
1) To get party's data, either we should have all the code in a class or
all
should be exposed as a service, to avoid confusion. Right now, it's
partially from the service (i.e. email & telephone) and partially from
the
ContactMechWorker class. All should be exposed as a service.
2) Also, as in the case of email & telephone, we can export the service,
no
service for postal address to export.
Does this makes sense ?, if yes, then I thought of making a service
wrapper
for ContactMechWorker.getCurrentPostalAddress &
ContactMechWorker.getPartyPostalAddresses, but then I noticed that one of
the parameter for these methods is ServletRequest object. In this case
what
can be done is create a service and can it from the methods above,
instead
of they hitting the delegator.
Please let me know your thoughts ? or am I missing something else ?
On Fri, Sep 18, 2009 at 2:54 AM, Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:
There are ContactMechWorker.getCurrentPostalAddress
ContactMechWorker.getPartyPostalAddresses
But yes no services, maybe because it's not needed ?
Jacques
From: "Abdullah Shaikh" <abdullah.sha...@viithiisys.com>
There are services to get the party telephone & email,
getPartyTelephone &
getPartyEmail respectively, but no service to get the party postal
address.
I can submit a patch for getPartyPostalAddress service, to get the
party's
postal address. What do you think ?
