You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ha...@apache.org on 2009/05/27 06:20:32 UTC

svn commit: r778988 - in /ofbiz/trunk/applications/party: data/PartyTypeData.xml webapp/partymgr/communication/CommForms.xml

Author: hansbak
Date: Wed May 27 04:20:31 2009
New Revision: 778988

URL: http://svn.apache.org/viewvc?rev=778988&view=rev
Log:
added comments to commevent status, removed the COM_READ status as COM_ROLE_READ status is now used

Modified:
    ofbiz/trunk/applications/party/data/PartyTypeData.xml
    ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml

Modified: ofbiz/trunk/applications/party/data/PartyTypeData.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/data/PartyTypeData.xml?rev=778988&r1=778987&r2=778988&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/data/PartyTypeData.xml (original)
+++ ofbiz/trunk/applications/party/data/PartyTypeData.xml Wed May 27 04:20:31 2009
@@ -300,27 +300,22 @@
     <StatusType description="Case" hasTable="N" parentTypeId="" statusTypeId="CASE_STATUS"/>
 
     <StatusType description="Communication Event" hasTable="N" parentTypeId="" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusItem description="Entered" sequenceId="01" statusCode="ENTERED" statusId="COM_ENTERED" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusItem description="Pending" sequenceId="02" statusCode="PENDING" statusId="COM_PENDING" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusItem description="Read" sequenceId="03" statusCode="READ" statusId="COM_READ" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusItem description="In-Progress" sequenceId="05" statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusItem description="Unknown Party" sequenceId="07" statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY" statusTypeId="COM_EVENT_STATUS"/>
+    <StatusItem description="Pending" sequenceId="01" statusCode="PENDING" statusId="COM_PENDING" statusTypeId="COM_EVENT_STATUS"/><!-- draft version, only visible by originator -->
+    <StatusItem description="Entered" sequenceId="02" statusCode="ENTERED" statusId="COM_ENTERED" statusTypeId="COM_EVENT_STATUS"/>
+    <!--StatusItem description="Read" sequenceId="03" statusCode="READ" statusId="COM_READ" statusTypeId="COM_EVENT_STATUS"/--><!-- the read status is maintained at the comm-role level; can be diffrent for all participants-->
+    <StatusItem description="In-Progress" sequenceId="05" statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS" statusTypeId="COM_EVENT_STATUS"/><!-- waiting to be sent -->
+    <StatusItem description="Unknown Party" sequenceId="07" statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY" statusTypeId="COM_EVENT_STATUS"/><!-- unknown incoming email address -->
     <StatusItem description="Closed" sequenceId="20" statusCode="COMPLETE" statusId="COM_COMPLETE" statusTypeId="COM_EVENT_STATUS"/>
     <StatusItem description="Resolved" sequenceId="21" statusCode="RESOLVED" statusId="COM_RESOLVED" statusTypeId="COM_EVENT_STATUS"/>
     <StatusItem description="Referred" sequenceId="22" statusCode="REFERRED" statusId="COM_REFERRED" statusTypeId="COM_EVENT_STATUS"/>
     <StatusItem description="Bounced" sequenceId="50" statusCode="BOUNCED" statusId="COM_BOUNCED" statusTypeId="COM_EVENT_STATUS"/>
     <StatusItem description="Cancelled" sequenceId="99" statusCode="CANCELLED" statusId="COM_CANCELLED" statusTypeId="COM_EVENT_STATUS"/>
-    <StatusValidChange condition="" statusId="COM_ENTERED" statusIdTo="COM_PENDING" transitionName="Set Pending"/>
-    <StatusValidChange condition="" statusId="COM_ENTERED" statusIdTo="COM_READ" transitionName="Read"/>
+    <StatusValidChange condition="" statusId="COM_ENTERED" statusIdTo="COM_PENDING" transitionName="Set Pending, only visible to originator"/>
     <StatusValidChange condition="" statusId="COM_ENTERED" statusIdTo="COM_COMPLETE" transitionName="Complete"/>
-    <StatusValidChange condition="" statusId="COM_PENDING" statusIdTo="COM_READ" transitionName="Read"/>
-    <StatusValidChange condition="" statusId="COM_PENDING" statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
-    <StatusValidChange condition="" statusId="COM_READ" statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
-    <StatusValidChange condition="" statusId="COM_READ" statusIdTo="COM_COMPLETE" transitionName="Complete"/>
-    <StatusValidChange condition="" statusId="COM_IN_PROGRESS" statusIdTo="COM_READ" transitionName="Read"/>
+    <StatusValidChange condition="" statusId="COM_PENDING" statusIdTo="COM_ENTERED" transitionName="Entered,visible to all participants"/>
+    <StatusValidChange condition="" statusId="COM_PENDING" statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress, waiting to be send"/>
     <StatusValidChange condition="" statusId="COM_IN_PROGRESS" statusIdTo="COM_COMPLETE" transitionName="Complete"/>
     <StatusValidChange condition="" statusId="COM_IN_PROGRESS" statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
-    <StatusValidChange condition="" statusId="COM_COMPLETE" statusIdTo="COM_READ" transitionName="Read"/>
     <StatusValidChange condition="" statusId="COM_COMPLETE" statusIdTo="COM_RESOLVED" transitionName="Resolve"/>
     <StatusValidChange condition="" statusId="COM_COMPLETE" statusIdTo="COM_REFERRED" transitionName="Refer"/>
     <StatusValidChange condition="" statusId="COM_COMPLETE" statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
@@ -330,7 +325,6 @@
 
     <StatusValidChange condition="" statusId="COM_ENTERED" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
     <StatusValidChange condition="" statusId="COM_PENDING" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
-    <StatusValidChange condition="" statusId="COM_READ" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
     <StatusValidChange condition="" statusId="COM_IN_PROGRESS" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
     <StatusValidChange condition="" statusId="COM_COMPLETE" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
     <StatusValidChange condition="" statusId="COM_RESOLVED" statusIdTo="COM_CANCELLED" transitionName="Cancel"/>

Modified: ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml?rev=778988&r1=778987&r2=778988&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml (original)
+++ ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml Wed May 27 04:20:31 2009
@@ -193,7 +193,7 @@
         <field name="send" title=" " position="1" use-when="communicationEvent!=null">
             <hyperlink also-hidden="true"
                 target-type="plain" description="${uiLabelMap.CommonSend}"
-                target="javascript:(document.EditInternalNote.statusId.value='COM_COMPLETE'),(document.EditInternalNote.datetimeStarted.value='${nowDate}'),(document.EditInternalNote.submit())"/>
+                target="javascript:(document.EditInternalNote.statusId.value='COM_ENTERED'),(document.EditInternalNote.datetimeStarted.value='${nowDate}'),(document.EditInternalNote.submit())"/>
         </field>
         <field name="save" title=" " position="2" use-when="communicationEvent!=null">
             <hyperlink also-hidden="true" target-type="plain" description="${uiLabelMap.CommonSave}" target="javascript:(document.EditInternalNote.submit())"/>



Re: svn commit: r778988 - in /ofbiz/trunk/applications/party: data/PartyTypeData.xml webapp/partymgr/communication/CommForms.xml

Posted by Andrew Zeneski <an...@hotwaxmedia.com>.
Sure Hans,

I documented this in rev 774634 but here it is again:

"Implemented a little event which marks a communication event as read,  
and returns a 1px gif image (spacer.gif) to the browser/mail client;  
include the following in your communication email:

<img src="http://localhost:8080/partymgr/control/ceimages/$ 
{communicationEventId}/logo.gif"/>

and when the message is opened, the event will be marked as read  
(assuming images are loaded); logo.gif can be any name, everything  
after the communication event ID in pathInfo is ignored "

So to test this just call the event, like you did but add the path info:
http://localhost:8080/partymgr/control/ceimages/[communication event  
Id]/logo.gif

I'll fix the NPE, so it just does nothing with out a communication  
event ID. What this should do is mark the recipient role as read.


Andrew

On Jun 4, 2009, at 12:35 AM, Hans Bakker wrote:

> Hi Andy,
>
> thanks for reporting this. I should have found this, sorry about that.
>
> I have now updated the e-commerce component in rev 781649 and add the
> messages to the top menu to make this more visible.
>
> One thing i did update and could not test very well is
> communicationEventServices.java, there is service which is called by  
> an
> external request and is called: markCommunicationAsRead
>
> i tried to do the request:
> http://localhost:8080/partymgr/control/ceimages
> but get a null pointer exception at line 1223
>
> perhaps you can explain how this service should work and perhaps can
> help here?
>
> Regards,
> Hans
>
>
> On Wed, 2009-06-03 at 13:39 -0400, Andrew Zeneski wrote:
>> Hans,
>>
>> This change has broken the functionality which marks outgoing (email)
>> communication events as read when the user opens the message. The
>> functionality uses the markCommEventRead which still references the
>> COM_READ status. Before removing something from a data file, please
>> make sure you have searched and updated all the code which references
>> that data.
>>
>> Doing a quick search for COM_ROLE_READ I am not finding any logic
>> which ties this in yet. So, either this service needs to be updated,
>> or we need to add back in the COM_READ status and the
>> StatusValidChange records.
>>
>> Please let me know your plans for this.
>>
>> Andrew
>>
>> On May 27, 2009, at 12:20 AM, hansbak@apache.org wrote:
>>
>>> Author: hansbak
>>> Date: Wed May 27 04:20:31 2009
>>> New Revision: 778988
>>>
>>> URL: http://svn.apache.org/viewvc?rev=778988&view=rev
>>> Log:
>>> added comments to commevent status, removed the COM_READ status as
>>> COM_ROLE_READ status is now used
>>>
>>> Modified:
>>>   ofbiz/trunk/applications/party/data/PartyTypeData.xml
>>>   ofbiz/trunk/applications/party/webapp/partymgr/communication/
>>> CommForms.xml
>>>
>>> Modified: ofbiz/trunk/applications/party/data/PartyTypeData.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/data/PartyTypeData.xml?rev=778988&r1=778987&r2=778988&view=diff
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> = 
>>> = 
>>> ====================================================================
>>> --- ofbiz/trunk/applications/party/data/PartyTypeData.xml (original)
>>> +++ ofbiz/trunk/applications/party/data/PartyTypeData.xml Wed May 27
>>> 04:20:31 2009
>>> @@ -300,27 +300,22 @@
>>>    <StatusType description="Case" hasTable="N" parentTypeId=""
>>> statusTypeId="CASE_STATUS"/>
>>>
>>>    <StatusType description="Communication Event" hasTable="N"
>>> parentTypeId="" statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusItem description="Entered" sequenceId="01"
>>> statusCode="ENTERED" statusId="COM_ENTERED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusItem description="Pending" sequenceId="02"
>>> statusCode="PENDING" statusId="COM_PENDING"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusItem description="Read" sequenceId="03"
>>> statusCode="READ" statusId="COM_READ"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusItem description="In-Progress" sequenceId="05"
>>> statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusItem description="Unknown Party" sequenceId="07"
>>> statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> +    <StatusItem description="Pending" sequenceId="01"
>>> statusCode="PENDING" statusId="COM_PENDING"
>>> statusTypeId="COM_EVENT_STATUS"/><!-- draft version, only visible by
>>> originator -->
>>> +    <StatusItem description="Entered" sequenceId="02"
>>> statusCode="ENTERED" statusId="COM_ENTERED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> +    <!--StatusItem description="Read" sequenceId="03"
>>> statusCode="READ" statusId="COM_READ"
>>> statusTypeId="COM_EVENT_STATUS"/--><!-- the read status is
>>> maintained at the comm-role level; can be diffrent for all
>>> participants-->
>>> +    <StatusItem description="In-Progress" sequenceId="05"
>>> statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"
>>> statusTypeId="COM_EVENT_STATUS"/><!-- waiting to be sent -->
>>> +    <StatusItem description="Unknown Party" sequenceId="07"
>>> statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"
>>> statusTypeId="COM_EVENT_STATUS"/><!-- unknown incoming email address
>>> -->
>>>    <StatusItem description="Closed" sequenceId="20"
>>> statusCode="COMPLETE" statusId="COM_COMPLETE"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>>    <StatusItem description="Resolved" sequenceId="21"
>>> statusCode="RESOLVED" statusId="COM_RESOLVED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>>    <StatusItem description="Referred" sequenceId="22"
>>> statusCode="REFERRED" statusId="COM_REFERRED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>>    <StatusItem description="Bounced" sequenceId="50"
>>> statusCode="BOUNCED" statusId="COM_BOUNCED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>>    <StatusItem description="Cancelled" sequenceId="99"
>>> statusCode="CANCELLED" statusId="COM_CANCELLED"
>>> statusTypeId="COM_EVENT_STATUS"/>
>>> -    <StatusValidChange condition="" statusId="COM_ENTERED"
>>> statusIdTo="COM_PENDING" transitionName="Set Pending"/>
>>> -    <StatusValidChange condition="" statusId="COM_ENTERED"
>>> statusIdTo="COM_READ" transitionName="Read"/>
>>> +    <StatusValidChange condition="" statusId="COM_ENTERED"
>>> statusIdTo="COM_PENDING" transitionName="Set Pending, only visible
>>> to originator"/>
>>>    <StatusValidChange condition="" statusId="COM_ENTERED"
>>> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
>>> -    <StatusValidChange condition="" statusId="COM_PENDING"
>>> statusIdTo="COM_READ" transitionName="Read"/>
>>> -    <StatusValidChange condition="" statusId="COM_PENDING"
>>> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
>>> -    <StatusValidChange condition="" statusId="COM_READ"
>>> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
>>> -    <StatusValidChange condition="" statusId="COM_READ"
>>> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
>>> -    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"
>>> statusIdTo="COM_READ" transitionName="Read"/>
>>> +    <StatusValidChange condition="" statusId="COM_PENDING"
>>> statusIdTo="COM_ENTERED" transitionName="Entered,visible to all
>>> participants"/>
>>> +    <StatusValidChange condition="" statusId="COM_PENDING"
>>> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress,
>>> waiting to be send"/>
>>>    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"
>>> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
>>>    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"
>>> statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
>>> -    <StatusValidChange condition="" statusId="COM_COMPLETE"
>>> statusIdTo="COM_READ" transitionName="Read"/>
>>>    <StatusValidChange condition="" statusId="COM_COMPLETE"
>>> statusIdTo="COM_RESOLVED" transitionName="Resolve"/>
>>>    <StatusValidChange condition="" statusId="COM_COMPLETE"
>>> statusIdTo="COM_REFERRED" transitionName="Refer"/>
>>>    <StatusValidChange condition="" statusId="COM_COMPLETE"
>>> statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
>>> @@ -330,7 +325,6 @@
>>>
>>>    <StatusValidChange condition="" statusId="COM_ENTERED"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>>    <StatusValidChange condition="" statusId="COM_PENDING"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>> -    <StatusValidChange condition="" statusId="COM_READ"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>>    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>>    <StatusValidChange condition="" statusId="COM_COMPLETE"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>>    <StatusValidChange condition="" statusId="COM_RESOLVED"
>>> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>>>
>>> Modified: ofbiz/trunk/applications/party/webapp/partymgr/
>>> communication/CommForms.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml?rev=778988&r1=778987&r2=778988&view=diff
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> = 
>>> = 
>>> ====================================================================
>>> --- ofbiz/trunk/applications/party/webapp/partymgr/communication/
>>> CommForms.xml (original)
>>> +++ ofbiz/trunk/applications/party/webapp/partymgr/communication/
>>> CommForms.xml Wed May 27 04:20:31 2009
>>> @@ -193,7 +193,7 @@
>>>        <field name="send" title=" " position="1" use-
>>> when="communicationEvent!=null">
>>>            <hyperlink also-hidden="true"
>>>                target-type="plain" description="$
>>> {uiLabelMap.CommonSend}"
>>> -                target="javascript:
>>> (document.EditInternalNote.statusId.value='COM_COMPLETE'),
>>> (document.EditInternalNote.datetimeStarted.value='${nowDate}'),
>>> (document.EditInternalNote.submit())"/>
>>> +                target="javascript:
>>> (document.EditInternalNote.statusId.value='COM_ENTERED'),
>>> (document.EditInternalNote.datetimeStarted.value='${nowDate}'),
>>> (document.EditInternalNote.submit())"/>
>>>        </field>
>>>        <field name="save" title=" " position="2" use-
>>> when="communicationEvent!=null">
>>>            <hyperlink also-hidden="true" target-type="plain"
>>> description="${uiLabelMap.CommonSave}" target="javascript:
>>> (document.EditInternalNote.submit())"/>
>>>
>>>
>>
> -- 
> Antwebsystems.com: Quality OFBiz services for competitive rates
>


Re: svn commit: r778988 - in /ofbiz/trunk/applications/party: data/PartyTypeData.xml webapp/partymgr/communication/CommForms.xml

Posted by Hans Bakker <ma...@antwebsystems.com>.
Hi Andy,

thanks for reporting this. I should have found this, sorry about that.

I have now updated the e-commerce component in rev 781649 and add the
messages to the top menu to make this more visible.

One thing i did update and could not test very well is
communicationEventServices.java, there is service which is called by an
external request and is called: markCommunicationAsRead

i tried to do the request: 
http://localhost:8080/partymgr/control/ceimages
but get a null pointer exception at line 1223

perhaps you can explain how this service should work and perhaps can
help here?

Regards,
Hans


On Wed, 2009-06-03 at 13:39 -0400, Andrew Zeneski wrote:
> Hans,
> 
> This change has broken the functionality which marks outgoing (email)  
> communication events as read when the user opens the message. The  
> functionality uses the markCommEventRead which still references the  
> COM_READ status. Before removing something from a data file, please  
> make sure you have searched and updated all the code which references  
> that data.
> 
> Doing a quick search for COM_ROLE_READ I am not finding any logic  
> which ties this in yet. So, either this service needs to be updated,  
> or we need to add back in the COM_READ status and the  
> StatusValidChange records.
> 
> Please let me know your plans for this.
> 
> Andrew
> 
> On May 27, 2009, at 12:20 AM, hansbak@apache.org wrote:
> 
> > Author: hansbak
> > Date: Wed May 27 04:20:31 2009
> > New Revision: 778988
> >
> > URL: http://svn.apache.org/viewvc?rev=778988&view=rev
> > Log:
> > added comments to commevent status, removed the COM_READ status as  
> > COM_ROLE_READ status is now used
> >
> > Modified:
> >    ofbiz/trunk/applications/party/data/PartyTypeData.xml
> >    ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> > CommForms.xml
> >
> > Modified: ofbiz/trunk/applications/party/data/PartyTypeData.xml
> > URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/data/PartyTypeData.xml?rev=778988&r1=778987&r2=778988&view=diff
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > ======================================================================
> > --- ofbiz/trunk/applications/party/data/PartyTypeData.xml (original)
> > +++ ofbiz/trunk/applications/party/data/PartyTypeData.xml Wed May 27  
> > 04:20:31 2009
> > @@ -300,27 +300,22 @@
> >     <StatusType description="Case" hasTable="N" parentTypeId=""  
> > statusTypeId="CASE_STATUS"/>
> >
> >     <StatusType description="Communication Event" hasTable="N"  
> > parentTypeId="" statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusItem description="Entered" sequenceId="01"  
> > statusCode="ENTERED" statusId="COM_ENTERED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusItem description="Pending" sequenceId="02"  
> > statusCode="PENDING" statusId="COM_PENDING"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusItem description="Read" sequenceId="03"  
> > statusCode="READ" statusId="COM_READ"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusItem description="In-Progress" sequenceId="05"  
> > statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusItem description="Unknown Party" sequenceId="07"  
> > statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > +    <StatusItem description="Pending" sequenceId="01"  
> > statusCode="PENDING" statusId="COM_PENDING"  
> > statusTypeId="COM_EVENT_STATUS"/><!-- draft version, only visible by  
> > originator -->
> > +    <StatusItem description="Entered" sequenceId="02"  
> > statusCode="ENTERED" statusId="COM_ENTERED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > +    <!--StatusItem description="Read" sequenceId="03"  
> > statusCode="READ" statusId="COM_READ"  
> > statusTypeId="COM_EVENT_STATUS"/--><!-- the read status is  
> > maintained at the comm-role level; can be diffrent for all  
> > participants-->
> > +    <StatusItem description="In-Progress" sequenceId="05"  
> > statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"  
> > statusTypeId="COM_EVENT_STATUS"/><!-- waiting to be sent -->
> > +    <StatusItem description="Unknown Party" sequenceId="07"  
> > statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"  
> > statusTypeId="COM_EVENT_STATUS"/><!-- unknown incoming email address  
> > -->
> >     <StatusItem description="Closed" sequenceId="20"  
> > statusCode="COMPLETE" statusId="COM_COMPLETE"  
> > statusTypeId="COM_EVENT_STATUS"/>
> >     <StatusItem description="Resolved" sequenceId="21"  
> > statusCode="RESOLVED" statusId="COM_RESOLVED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> >     <StatusItem description="Referred" sequenceId="22"  
> > statusCode="REFERRED" statusId="COM_REFERRED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> >     <StatusItem description="Bounced" sequenceId="50"  
> > statusCode="BOUNCED" statusId="COM_BOUNCED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> >     <StatusItem description="Cancelled" sequenceId="99"  
> > statusCode="CANCELLED" statusId="COM_CANCELLED"  
> > statusTypeId="COM_EVENT_STATUS"/>
> > -    <StatusValidChange condition="" statusId="COM_ENTERED"  
> > statusIdTo="COM_PENDING" transitionName="Set Pending"/>
> > -    <StatusValidChange condition="" statusId="COM_ENTERED"  
> > statusIdTo="COM_READ" transitionName="Read"/>
> > +    <StatusValidChange condition="" statusId="COM_ENTERED"  
> > statusIdTo="COM_PENDING" transitionName="Set Pending, only visible  
> > to originator"/>
> >     <StatusValidChange condition="" statusId="COM_ENTERED"  
> > statusIdTo="COM_COMPLETE" transitionName="Complete"/>
> > -    <StatusValidChange condition="" statusId="COM_PENDING"  
> > statusIdTo="COM_READ" transitionName="Read"/>
> > -    <StatusValidChange condition="" statusId="COM_PENDING"  
> > statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
> > -    <StatusValidChange condition="" statusId="COM_READ"  
> > statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
> > -    <StatusValidChange condition="" statusId="COM_READ"  
> > statusIdTo="COM_COMPLETE" transitionName="Complete"/>
> > -    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> > statusIdTo="COM_READ" transitionName="Read"/>
> > +    <StatusValidChange condition="" statusId="COM_PENDING"  
> > statusIdTo="COM_ENTERED" transitionName="Entered,visible to all  
> > participants"/>
> > +    <StatusValidChange condition="" statusId="COM_PENDING"  
> > statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress,  
> > waiting to be send"/>
> >     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> > statusIdTo="COM_COMPLETE" transitionName="Complete"/>
> >     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> > statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
> > -    <StatusValidChange condition="" statusId="COM_COMPLETE"  
> > statusIdTo="COM_READ" transitionName="Read"/>
> >     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> > statusIdTo="COM_RESOLVED" transitionName="Resolve"/>
> >     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> > statusIdTo="COM_REFERRED" transitionName="Refer"/>
> >     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> > statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
> > @@ -330,7 +325,6 @@
> >
> >     <StatusValidChange condition="" statusId="COM_ENTERED"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> >     <StatusValidChange condition="" statusId="COM_PENDING"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> > -    <StatusValidChange condition="" statusId="COM_READ"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> >     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> >     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> >     <StatusValidChange condition="" statusId="COM_RESOLVED"  
> > statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> >
> > Modified: ofbiz/trunk/applications/party/webapp/partymgr/ 
> > communication/CommForms.xml
> > URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml?rev=778988&r1=778987&r2=778988&view=diff
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > = 
> > ======================================================================
> > --- ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> > CommForms.xml (original)
> > +++ ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> > CommForms.xml Wed May 27 04:20:31 2009
> > @@ -193,7 +193,7 @@
> >         <field name="send" title=" " position="1" use- 
> > when="communicationEvent!=null">
> >             <hyperlink also-hidden="true"
> >                 target-type="plain" description="$ 
> > {uiLabelMap.CommonSend}"
> > -                target="javascript: 
> > (document.EditInternalNote.statusId.value='COM_COMPLETE'), 
> > (document.EditInternalNote.datetimeStarted.value='${nowDate}'), 
> > (document.EditInternalNote.submit())"/>
> > +                target="javascript: 
> > (document.EditInternalNote.statusId.value='COM_ENTERED'), 
> > (document.EditInternalNote.datetimeStarted.value='${nowDate}'), 
> > (document.EditInternalNote.submit())"/>
> >         </field>
> >         <field name="save" title=" " position="2" use- 
> > when="communicationEvent!=null">
> >             <hyperlink also-hidden="true" target-type="plain"  
> > description="${uiLabelMap.CommonSave}" target="javascript: 
> > (document.EditInternalNote.submit())"/>
> >
> >
> 
-- 
Antwebsystems.com: Quality OFBiz services for competitive rates


Re: svn commit: r778988 - in /ofbiz/trunk/applications/party: data/PartyTypeData.xml webapp/partymgr/communication/CommForms.xml

Posted by Andrew Zeneski <an...@hotwaxmedia.com>.
Hans,

This change has broken the functionality which marks outgoing (email)  
communication events as read when the user opens the message. The  
functionality uses the markCommEventRead which still references the  
COM_READ status. Before removing something from a data file, please  
make sure you have searched and updated all the code which references  
that data.

Doing a quick search for COM_ROLE_READ I am not finding any logic  
which ties this in yet. So, either this service needs to be updated,  
or we need to add back in the COM_READ status and the  
StatusValidChange records.

Please let me know your plans for this.

Andrew

On May 27, 2009, at 12:20 AM, hansbak@apache.org wrote:

> Author: hansbak
> Date: Wed May 27 04:20:31 2009
> New Revision: 778988
>
> URL: http://svn.apache.org/viewvc?rev=778988&view=rev
> Log:
> added comments to commevent status, removed the COM_READ status as  
> COM_ROLE_READ status is now used
>
> Modified:
>    ofbiz/trunk/applications/party/data/PartyTypeData.xml
>    ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> CommForms.xml
>
> Modified: ofbiz/trunk/applications/party/data/PartyTypeData.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/data/PartyTypeData.xml?rev=778988&r1=778987&r2=778988&view=diff
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- ofbiz/trunk/applications/party/data/PartyTypeData.xml (original)
> +++ ofbiz/trunk/applications/party/data/PartyTypeData.xml Wed May 27  
> 04:20:31 2009
> @@ -300,27 +300,22 @@
>     <StatusType description="Case" hasTable="N" parentTypeId=""  
> statusTypeId="CASE_STATUS"/>
>
>     <StatusType description="Communication Event" hasTable="N"  
> parentTypeId="" statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusItem description="Entered" sequenceId="01"  
> statusCode="ENTERED" statusId="COM_ENTERED"  
> statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusItem description="Pending" sequenceId="02"  
> statusCode="PENDING" statusId="COM_PENDING"  
> statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusItem description="Read" sequenceId="03"  
> statusCode="READ" statusId="COM_READ"  
> statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusItem description="In-Progress" sequenceId="05"  
> statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"  
> statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusItem description="Unknown Party" sequenceId="07"  
> statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"  
> statusTypeId="COM_EVENT_STATUS"/>
> +    <StatusItem description="Pending" sequenceId="01"  
> statusCode="PENDING" statusId="COM_PENDING"  
> statusTypeId="COM_EVENT_STATUS"/><!-- draft version, only visible by  
> originator -->
> +    <StatusItem description="Entered" sequenceId="02"  
> statusCode="ENTERED" statusId="COM_ENTERED"  
> statusTypeId="COM_EVENT_STATUS"/>
> +    <!--StatusItem description="Read" sequenceId="03"  
> statusCode="READ" statusId="COM_READ"  
> statusTypeId="COM_EVENT_STATUS"/--><!-- the read status is  
> maintained at the comm-role level; can be diffrent for all  
> participants-->
> +    <StatusItem description="In-Progress" sequenceId="05"  
> statusCode="IN_PROGRESS" statusId="COM_IN_PROGRESS"  
> statusTypeId="COM_EVENT_STATUS"/><!-- waiting to be sent -->
> +    <StatusItem description="Unknown Party" sequenceId="07"  
> statusCode="UNKNOWN_PARTY" statusId="COM_UNKNOWN_PARTY"  
> statusTypeId="COM_EVENT_STATUS"/><!-- unknown incoming email address  
> -->
>     <StatusItem description="Closed" sequenceId="20"  
> statusCode="COMPLETE" statusId="COM_COMPLETE"  
> statusTypeId="COM_EVENT_STATUS"/>
>     <StatusItem description="Resolved" sequenceId="21"  
> statusCode="RESOLVED" statusId="COM_RESOLVED"  
> statusTypeId="COM_EVENT_STATUS"/>
>     <StatusItem description="Referred" sequenceId="22"  
> statusCode="REFERRED" statusId="COM_REFERRED"  
> statusTypeId="COM_EVENT_STATUS"/>
>     <StatusItem description="Bounced" sequenceId="50"  
> statusCode="BOUNCED" statusId="COM_BOUNCED"  
> statusTypeId="COM_EVENT_STATUS"/>
>     <StatusItem description="Cancelled" sequenceId="99"  
> statusCode="CANCELLED" statusId="COM_CANCELLED"  
> statusTypeId="COM_EVENT_STATUS"/>
> -    <StatusValidChange condition="" statusId="COM_ENTERED"  
> statusIdTo="COM_PENDING" transitionName="Set Pending"/>
> -    <StatusValidChange condition="" statusId="COM_ENTERED"  
> statusIdTo="COM_READ" transitionName="Read"/>
> +    <StatusValidChange condition="" statusId="COM_ENTERED"  
> statusIdTo="COM_PENDING" transitionName="Set Pending, only visible  
> to originator"/>
>     <StatusValidChange condition="" statusId="COM_ENTERED"  
> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
> -    <StatusValidChange condition="" statusId="COM_PENDING"  
> statusIdTo="COM_READ" transitionName="Read"/>
> -    <StatusValidChange condition="" statusId="COM_PENDING"  
> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
> -    <StatusValidChange condition="" statusId="COM_READ"  
> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress"/>
> -    <StatusValidChange condition="" statusId="COM_READ"  
> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
> -    <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> statusIdTo="COM_READ" transitionName="Read"/>
> +    <StatusValidChange condition="" statusId="COM_PENDING"  
> statusIdTo="COM_ENTERED" transitionName="Entered,visible to all  
> participants"/>
> +    <StatusValidChange condition="" statusId="COM_PENDING"  
> statusIdTo="COM_IN_PROGRESS" transitionName="Set In Progress,  
> waiting to be send"/>
>     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> statusIdTo="COM_COMPLETE" transitionName="Complete"/>
>     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
> -    <StatusValidChange condition="" statusId="COM_COMPLETE"  
> statusIdTo="COM_READ" transitionName="Read"/>
>     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> statusIdTo="COM_RESOLVED" transitionName="Resolve"/>
>     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> statusIdTo="COM_REFERRED" transitionName="Refer"/>
>     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> statusIdTo="COM_BOUNCED" transitionName="Bounced"/>
> @@ -330,7 +325,6 @@
>
>     <StatusValidChange condition="" statusId="COM_ENTERED"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>     <StatusValidChange condition="" statusId="COM_PENDING"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
> -    <StatusValidChange condition="" statusId="COM_READ"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>     <StatusValidChange condition="" statusId="COM_IN_PROGRESS"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>     <StatusValidChange condition="" statusId="COM_COMPLETE"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>     <StatusValidChange condition="" statusId="COM_RESOLVED"  
> statusIdTo="COM_CANCELLED" transitionName="Cancel"/>
>
> Modified: ofbiz/trunk/applications/party/webapp/partymgr/ 
> communication/CommForms.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/webapp/partymgr/communication/CommForms.xml?rev=778988&r1=778987&r2=778988&view=diff
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> CommForms.xml (original)
> +++ ofbiz/trunk/applications/party/webapp/partymgr/communication/ 
> CommForms.xml Wed May 27 04:20:31 2009
> @@ -193,7 +193,7 @@
>         <field name="send" title=" " position="1" use- 
> when="communicationEvent!=null">
>             <hyperlink also-hidden="true"
>                 target-type="plain" description="$ 
> {uiLabelMap.CommonSend}"
> -                target="javascript: 
> (document.EditInternalNote.statusId.value='COM_COMPLETE'), 
> (document.EditInternalNote.datetimeStarted.value='${nowDate}'), 
> (document.EditInternalNote.submit())"/>
> +                target="javascript: 
> (document.EditInternalNote.statusId.value='COM_ENTERED'), 
> (document.EditInternalNote.datetimeStarted.value='${nowDate}'), 
> (document.EditInternalNote.submit())"/>
>         </field>
>         <field name="save" title=" " position="2" use- 
> when="communicationEvent!=null">
>             <hyperlink also-hidden="true" target-type="plain"  
> description="${uiLabelMap.CommonSave}" target="javascript: 
> (document.EditInternalNote.submit())"/>
>
>