04-07-2017, 01:59 AM,
(This post was last modified: 04-07-2017, 02:05 AM by dalescott.)
|
|
dalescott
Senior Member
|
Posts: 107
Threads: 23
Joined: Sep 2016
|
|
RE: Issues With Lists Using foreach($Result as $MyRow)
I'm just a user, what were the symptoms? SQL errors rendered in WebERP? What did you do that resulted in the symptoms? I grabbed a screenshot with SQL errors recently when working through the contracts process, but I haven't had a chance to revisit.
Also, and I don't mean any disrespect, but why not include a patch? Was it the project's fault for not explaining how? Waiting to see if someone asks for a patch? Just curious...
And really, I am not trying to call anyone out, I'm just curious because community feedback is how a project gets better. Also, if anyone PMs me I'll post an anonymized group summary.
Forgot PM is disabled. dale@dalescott.net
http://www.dalescott.net
|
|
04-07-2017, 02:11 AM,
(This post was last modified: 04-07-2017, 03:22 AM by VortecCPI.)
|
|
VortecCPI
Paul Becker
|
Posts: 751
Threads: 211
Joined: Apr 2017
|
|
RE: Issues With Lists Using foreach($Result as $MyRow)
The symptom is an empty list found in the pages listed above; no SQL errors.
Before:
After:
|
|
04-08-2017, 02:36 AM,
|
|
VortecCPI
Paul Becker
|
Posts: 751
Threads: 211
Joined: Apr 2017
|
|
RE: Issues With Lists Using foreach($Result as $MyRow)
(04-07-2017, 11:07 PM)falkoner Wrote: I would suggest you create a patch for this change and submit it to submissions@weberp.org
I should say this patch should be against the latest SVN code from Sourceforge.
Tim
While I have no issue with work related to doing this I think it is best left to the experts. Any changes should be adequately tested and verified for proper operation and I am not qualified for such work. While I am familiar with PHP and have been developing applications since 1983 I am NOT intimately familiar with webERP.
I would rather a qualified contributor/developer manage any changes to a released version of webERP.
|
|
|