Connection to AP/AR apps

[Erwan de FERRIERES]

Hi all,

the connection to AP/AR applications is no more possible.

Steps to reproduce :
upgrade trunk, compile, start and connect to ap or ar

error message :
ERROR rendering error page [/error/error.jsp], but here is the error
text: org.ofbiz.widget.screen.ScreenRenderException: Error rendering
screen [component://accounting/widget/ap/CommonScreens.xml#main]:
org.ofbiz.base.util.GeneralException: Error running Groovy script at
location
[component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
(The current transaction is marked for rollback, not beginning a new
transaction and aborting current operation; the rollbackOnly was caused
by: Service [getPartyAccountingPreferences] threw an unexpected
exception/errororg.ofbiz.service.ServiceValidationException: The
following required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId] (The following
required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId])) (Error running
Groovy script at location
[component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
(The current transaction is marked for rollback, not beginning a new
transaction and aborting current operation; the rollbackOnly was caused
by: Service [getPartyAccountingPreferences] threw an unexpected
exception/errororg.ofbiz.service.ServiceValidationException: The
following required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId] (The following
required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId])))

Cheers,

--
Erwan de FERRIERES
www.nereide.biz

[Jacques Le Roux]

Erwan: could it be related to http://svn.apache.org/viewvc?rev=917790&view=rev ?

Adam : is there a reason to keep OFBIZ-3513 and OFBIZ-3514 open?

Jacques

From: "Erwan de FERRIERES" <[hidden email]>

> Hi all,
>
> the connection to AP/AR applications is no more possible.
>
> Steps to reproduce :
> upgrade trunk, compile, start and connect to ap or ar
>
> error message :
> ERROR rendering error page [/error/error.jsp], but here is the error
> text: org.ofbiz.widget.screen.ScreenRenderException: Error rendering
> screen [component://accounting/widget/ap/CommonScreens.xml#main]:
> org.ofbiz.base.util.GeneralException: Error running Groovy script at
> location
> [component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
> (The current transaction is marked for rollback, not beginning a new
> transaction and aborting current operation; the rollbackOnly was caused
> by: Service [getPartyAccountingPreferences] threw an unexpected
> exception/errororg.ofbiz.service.ServiceValidationException: The
> following required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId] (The following
> required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId])) (Error running
> Groovy script at location
> [component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
> (The current transaction is marked for rollback, not beginning a new
> transaction and aborting current operation; the rollbackOnly was caused
> by: Service [getPartyAccountingPreferences] threw an unexpected
> exception/errororg.ofbiz.service.ServiceValidationException: The
> following required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId] (The following
> required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId])))
>
> Cheers,
>
> --
> Erwan de FERRIERES
> www.nereide.biz
>

[Erwan de FERRIERES]

Le 02/03/2010 10:40, Jacques Le Roux a écrit :
> Erwan: could it be related to
> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>
Yes, at first sight, it may come from this.

> Adam : is there a reason to keep OFBIZ-3513 and OFBIZ-3514 open?
>
> Jacques
>

--
Erwan de FERRIERES
www.nereide.biz

[Adam Heath-2]

Erwan de FERRIERES wrote:

> Hi all,
>
> the connection to AP/AR applications is no more possible.
>
> Steps to reproduce :
> upgrade trunk, compile, start and connect to ap or ar
>
> error message :
> ERROR rendering error page [/error/error.jsp], but here is the error
> text: org.ofbiz.widget.screen.ScreenRenderException: Error rendering
> screen [component://accounting/widget/ap/CommonScreens.xml#main]:
> org.ofbiz.base.util.GeneralException: Error running Groovy script at
> location
> [component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
> (The current transaction is marked for rollback, not beginning a new
> transaction and aborting current operation; the rollbackOnly was caused
> by: Service [getPartyAccountingPreferences] threw an unexpected
> exception/errororg.ofbiz.service.ServiceValidationException: The
> following required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId] (The following
> required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId])) (Error running
> Groovy script at location
> [component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
> (The current transaction is marked for rollback, not beginning a new
> transaction and aborting current operation; the rollbackOnly was caused
> by: Service [getPartyAccountingPreferences] threw an unexpected
> exception/errororg.ofbiz.service.ServiceValidationException: The
> following required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId] (The following
> required parameter is missing:
> [getPartyAccountingPreferences.organizationPartyId])))

checking...

[Adam Heath-2]

Jacques Le Roux wrote:
> Erwan: could it be related to
> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>
> Adam : is there a reason to keep OFBIZ-3513 and OFBIZ-3514 open?

No.  Ean created those bugs, so that I could have a record of the
patch submission from him, so that I could apply to trunk, so I could
produce a new internal brainfood ofbiz deployment, and just hadn't
gotten around to closing them.

[Adam Heath-2]

Jacques Le Roux wrote:
> Erwan: could it be related to
> http://svn.apache.org/viewvc?rev=917790&view=rev ?

That's not possible.  Not only is that an internal ordermgr action
script, I just added a print stmt in the if block that I modified, and
it didn't print anything.

Time to do a git bisect.

[Jacques Le Roux]

From: "Adam Heath" <[hidden email]>
> Jacques Le Roux wrote:
>> Erwan: could it be related to
>> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>
> That's not possible.  Not only is that an internal ordermgr action
> script, I just added a print stmt in the if block that I modified, and
> it didn't print anything.
>
> Time to do a git bisect.

Sounds like magic :o)

Jacques

[Adam Heath-2]

Jacques Le Roux wrote:

> From: "Adam Heath" <[hidden email]>
>> Jacques Le Roux wrote:
>>> Erwan: could it be related to
>>> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>>
>> That's not possible.  Not only is that an internal ordermgr action
>> script, I just added a print stmt in the if block that I modified, and
>> it didn't print anything.
>>
>> Time to do a git bisect.
>
> Sounds like magic :o)

Yeah, it kinda is, the kind I like.

It's revision 917373, probably the change to FSE.

[Jacques Le Roux]

http://www.wandisco.com/php/pr.php?prdate=2010-02-17

The Subversion development community does not deny...

Jacques

From: "Adam Heath" <[hidden email]>

> Jacques Le Roux wrote:
>> From: "Adam Heath" <[hidden email]>
>>> Jacques Le Roux wrote:
>>>> Erwan: could it be related to
>>>> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>>>
>>> That's not possible.  Not only is that an internal ordermgr action
>>> script, I just added a print stmt in the if block that I modified, and
>>> it didn't print anything.
>>>
>>> Time to do a git bisect.
>>
>> Sounds like magic :o)
>
> Yeah, it kinda is, the kind I like.
>
> It's revision 917373, probably the change to FSE.
>

[Adam Heath-2]

Jacques Le Roux wrote:
> http://www.wandisco.com/php/pr.php?prdate=2010-02-17

Not one to complain, but that page is so lacking in details, even the
linked pages are bare of anything substantial.

Offline commits sound cool, and fixing the svn poop problem(scattered
.svn folders all over) is nice, there are lots and lots of other
things in git that are useful.

Having the entire previous history immediately available, is immensly
useful.  When someone reports a problem, I can use git bisect to find
the problem.  I don't have to wait for svn to download the deltas
between all old versions.

Same is true when I'm trying to just look at previous code.

Being able to have a checkout of trunk, with all my current uncommited
work, and any staged commits that haven't been pushed upstream, then
suddenly having to stop all work, to fix some critical bug in a
deployed version, is very nice.  I do a git stash(takes all modified
files and saves them temporarily).  Then, I do a git co
$old_production_release_branch.  Edit, commit, build deployment.  git
co trunk, git stash apply, and continue where I left off.

When we finally decide to upgrade $old_production_release, I check it
out, then do the normal git svn rebase command.  The last time I did
this, it only took me half a day to upgrade a 6 month old version.
That is vastly superior to the old way we did it(by hand).  It was
made simpler this time around, because I forced myself to put all
changes/fixes upstream first, before adding them to the old package.

[Adam Heath-2]

Adam Heath wrote:

> Jacques Le Roux wrote:
>> From: "Adam Heath" <[hidden email]>
>>> Jacques Le Roux wrote:
>>>> Erwan: could it be related to
>>>> http://svn.apache.org/viewvc?rev=917790&view=rev ?
>>> That's not possible.  Not only is that an internal ordermgr action
>>> script, I just added a print stmt in the if block that I modified, and
>>> it didn't print anything.
>>>
>>> Time to do a git bisect.
>> Sounds like magic :o)
>
> Yeah, it kinda is, the kind I like.
>
> It's revision 917373, probably the change to FSE.

Fixed this in 918769.