Login!
Lost password?
 

MODx Bug/Feature Tracker and Feature Requests

Welcome to the MODx CMS Tracker. Please choose the appropriate project from the drop down menu and provide as much information as possible regarding your server environment and browser. Thanks!

FS#934 — Bug in function getDocuments()

Attached to Project — MODx
Opened by Serg (Robust) - Thursday, 16 August 2007, 03:59AM
Last edited by Garry Nutting (garryn) - Monday, 19 November 2007, 10:01PM
Task Type Bug Report
Category Core Distribution
Status Closed
Assigned To Garry Nutting (garryn)
Operating System All
Severity Low
Priority Normal
Reported Version 0.9.6
Due in Version 0.9.6.2
Due Date Undecided
Percent Complete 100%

Details

PHP error debug
Error: join() [function.join]: Bad arguments.
Error type/ Nr.: Warning - 2
File: manager/includes/document.parser.class.inc.php
Line: 1411
Line 1411 source: WHERE (sc.id IN (" . join($ids, ",") . ") AND sc.published=$published AND sc.deleted=$deleted $where)


Replace join($ids, ",") to join(",",$ids)

This task depends upon

This task blocks these from closing
Closed by  Brunaud (coroico)
Friday, 09 May 2008, 08:47AM
Reason for closing:  Fixed
Comment by Garry Nutting (garryn) - Monday, 19 November 2007, 10:01PM
  • Field changed: Status (Unconfirmed → Requires testing)
  • Field changed: Due in Version (Undecided → 0.9.6.2)
  • Field changed: Percent Complete (0% → 100%)
  • Field changed: Due Date (Undecided → Undecided)
  • Task assigned to Garry Nutting (garryn)
Should be fixed in SVN rev 3163.

Comment by Matthew Pietz (sirlancelot) - Monday, 03 December 2007, 11:04AM
Is is just personal preference to use join() instead of implode() or is join() added for legacy?

Comment by Brunaud (coroico) - Friday, 09 May 2008, 08:46AM
As set up a user functional test is not obvious, I have simply checked the correction. Implode has been used rather his alias join