ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacques Le Roux (JIRA)" <j...@apache.org>
Subject [jira] [Created] (OFBIZ-9164) Refactor ContentWorkerInterface methods signatures
Date Sun, 01 Jan 2017 11:36:58 GMT
Jacques Le Roux created OFBIZ-9164:

             Summary: Refactor ContentWorkerInterface methods signatures
                 Key: OFBIZ-9164
                 URL: https://issues.apache.org/jira/browse/OFBIZ-9164
             Project: OFBiz
          Issue Type: Sub-task
          Components: content, framework, order, party, product, specialpurpose/lucene, workeffort
    Affects Versions: Trunk
            Reporter: Jacques Le Roux
            Assignee: Jacques Le Roux
            Priority: Minor
             Fix For: Upcoming Release

While working on OFBIZ-6919 which was built on R13.07 I stumbled upon an issue due to r1652852
where Adrian improved the cacheKey in FormFactory.getFormFromLocation() by adding a delegator
reference (Tenants). Actually I'm not even sure it was done at r1652852 because Adrian did
not maintain the FormFactory svn history.

Anyway, to make a long story short I had to introduce a DispatchContext parameter when calling
FormFactory.readFormDocument() when the code from R13.07 only passed a null.

This had an impact in the hierarchy tree because FormFactory.readFormDocument() was called
in DataResourceWorker class, where the new code was called from renderDataResourceAsText().
So I instead of only passing a Delegator I decided to pass only a LocalDispatcher parameter
in renderDataResourceAsText(), since we can get the Delegator  from the LocalDispatcher. Doing
so it had an impact on the renderDataResourceAsText hierarchy tree ending in DataResourceWorkerInterface
and all related.

 I finally decided to apply the same ["Change Method Signature" refactoring pattern|http://refactoring.com/catalog/addParameter.html]
to all cases related to ContentWorkerInterface. No need to pass a delegator when you have

Here I attach a patch for review, I'll commit in few days

This message was sent by Atlassian JIRA

View raw message