| std.update3.inc confusion [message #2365] |
Sat, 14 November 2009 11:30  |
ljkbrost
Messages: 59
Registered: April 2006
|
Member |
|
|
Hi,
I'm trying to use the std.update3.inc pattern to create a dynamic report generation screen and I've run into a problem that I have not seen before and I'm not sure of the fix.
To give an idea of what I'm trying to do:
I have a rpt_report and rpt_parameter table. The rpt_report table contains the description of the report and the rpt_parameter table contains the description of the parameters (one parameter per row).
1) Display list1 of the reports
2) Select a report from the list1 screen.
3) Click the run_report (an update3 screen)
4) Inside the _cm_changeConfig fuction:
4.1) Grab a list of all the parameters based on the passed in report_id (populated by the list1->update3 selection process)
4.2) Build the fieldspec based on the data pulled in from this report.
So this is all pretty simple/straight forward, but I have a problem the somehow my where clause in the query to get the parameters has an extra '=' operator.
Query:
SELECT *
FROM
rpt_report
JOIN
rpt_version
ON (rpt_report.report_default_version_id =
rpt_version.version_id)
RIGHT JOIN
rpt_parameter
USING (version_id)
WHERE
rpt_report.report_id='='060a-7bb8-36e67021fcb81fe-01224c''
Notice the WHERE clause has an extra "=" in the front.
I've traced the problem to the std.table.class.inc's initialise() method. Inside this class it there is the following call:
// convert $where string to an array
$fieldarray = where2array($where, false, false);
This function ultimately returns a fieldarray with
Array
(
[report_id] => ='060a-7bb8-36e67021fcb81fe-01224c'
)
Stepping through the where2array function everything works upto the point when the function strips the operators. In this case the function is passed $strip_operators=false, so it leaves the extra '=' in the fieldarray.
Ultimately when I'm in _cm_changeConfig, my $fieldarray is not what I would expect. Changing the initialise() method to strip the operators fixes the problem.
So, should the where2array be change to strip_operators=true?
Or, am I missing something?
|
|
|
|
| Re: std.update3.inc confusion [message #2366 is a reply to message #2365] |
Sat, 14 November 2009 13:31  |
AJM
Messages: 2368
Registered: April 2006
Location: Surrey, UK
|
Senior Member |
|
|
The call to where2array() in the initialise() method should NOT be changed to set stripoperators=true as the $where string passed down from the previous object may have operators such as IN, BETWEEN, <, >, etc, so it is important that they are NOT stripped as they could not be reconstituted when the array is converted back into a string.
You will need to call the stripOperators() function in the relevant custom method.
Tony Marston
http://www.tonymarston.net
http://www.radicore.org
|
|
|
|
| Re: std.update3.inc confusion [message #2367 is a reply to message #2366] |
Sat, 14 November 2009 14:43 |
ljkbrost
Messages: 59
Registered: April 2006
|
Member |
|
|
Thank you for your opinion on the matter. I've abandoned this approach as the std.update3.inc is too specific to your implementation of general control settings.
I'm going to look at using a simplified add1 or update1 to achieve my needs.
Cheers,
Kyle Brost
----
www.softelephant.com
|
|
|
|
Radicore Forum
Search
Help
Members
Register
Login
Home
