You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by as...@apache.org on 2008/06/24 14:49:40 UTC

svn commit: r671158 - /ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh

Author: ashish
Date: Tue Jun 24 05:49:39 2008
New Revision: 671158

URL: http://svn.apache.org/viewvc?rev=671158&view=rev
Log:
I think we will read the partyId from the timeSheet generic value.
I noticed that we are not passing partyId (From the respective Form) to services "updateTimesheet" & "updateTimeEntryByWorkeffort".

If there is some reason for not passing partyId to the above services then please let us know.
Because we are considering that the "partyId" field from TimeEntry will contain the record entered by "Logged In" party.

Modified:
    ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh

Modified: ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh?rev=671158&r1=671157&r2=671158&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh (original)
+++ ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh Tue Jun 24 05:49:39 2008
@@ -49,7 +49,7 @@
                     // check the role of his part, if party is a provider
                     exprs = UtilMisc.toList(
                                 EntityCondition.makeCondition("workEffortId", EntityOperator.EQUALS, timeEntry.getString("workEffortId")),
-                                EntityCondition.makeCondition("partyId", EntityOperator.EQUALS, timeEntry.getString("partyId")),
+                                EntityCondition.makeCondition("partyId", EntityOperator.EQUALS, timesheet.getString("partyId")),
                                 EntityCondition.makeCondition("roleTypeId", EntityOperator.LIKE, "PROVIDER_%"));
                     ecl = EntityCondition.makeCondition(exprs, EntityOperator.AND);
                     providerRoles = EntityUtil.filterByDate(delegator.findList("WorkEffortPartyAssignment", ecl, null, null, null, false));



Re: svn commit: r671158 - /ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh

Posted by Ashish Vijaywargiya <vi...@gmail.com>.
Thanks Ratnesh Upadhyay for discussing this issue with me :-).

--
Ashish

On Tue, Jun 24, 2008 at 8:49 AM, <as...@apache.org> wrote:

> Author: ashish
> Date: Tue Jun 24 05:49:39 2008
> New Revision: 671158
>
> URL: http://svn.apache.org/viewvc?rev=671158&view=rev
> Log:
> I think we will read the partyId from the timeSheet generic value.
> I noticed that we are not passing partyId (From the respective Form) to
> services "updateTimesheet" & "updateTimeEntryByWorkeffort".
>
> If there is some reason for not passing partyId to the above services then
> please let us know.
> Because we are considering that the "partyId" field from TimeEntry will
> contain the record entered by "Logged In" party.
>
> Modified:
>
>  ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh
>
> Modified:
> ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh?rev=671158&r1=671157&r2=671158&view=diff
>
> ==============================================================================
> ---
> ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh
> (original)
> +++
> ofbiz/trunk/specialpurpose/projectmgr/webapp/projectmgr/WEB-INF/actions/ListUnBilledHoursResources.bsh
> Tue Jun 24 05:49:39 2008
> @@ -49,7 +49,7 @@
>                     // check the role of his part, if party is a provider
>                     exprs = UtilMisc.toList(
>
> EntityCondition.makeCondition("workEffortId", EntityOperator.EQUALS,
> timeEntry.getString("workEffortId")),
> -                                EntityCondition.makeCondition("partyId",
> EntityOperator.EQUALS, timeEntry.getString("partyId")),
> +                                EntityCondition.makeCondition("partyId",
> EntityOperator.EQUALS, timesheet.getString("partyId")),
>                                 EntityCondition.makeCondition("roleTypeId",
> EntityOperator.LIKE, "PROVIDER_%"));
>                     ecl = EntityCondition.makeCondition(exprs,
> EntityOperator.AND);
>                     providerRoles =
> EntityUtil.filterByDate(delegator.findList("WorkEffortPartyAssignment", ecl,
> null, null, null, false));
>
>
>