Skip to content

Commit b1cde1f

Browse files
committed
Type-hint csv_from_result(), xml_from_result() DB utilities
Close bcit-ci#4299
1 parent c83cfe8 commit b1cde1f

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

system/database/DB_utility.php

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -235,13 +235,8 @@ public function repair_table($table_name)
235235
* @param string $enclosure Enclosure (default: ")
236236
* @return string
237237
*/
238-
public function csv_from_result($query, $delim = ',', $newline = "\n", $enclosure = '"')
238+
public function csv_from_result(CI_DB_result $query, $delim = ',', $newline = "\n", $enclosure = '"')
239239
{
240-
if ( ! is_object($query) OR ! method_exists($query, 'list_fields'))
241-
{
242-
show_error('You must submit a valid result object');
243-
}
244-
245240
$out = '';
246241
// First generate the headings from the table column names
247242
foreach ($query->list_fields() as $name)
@@ -274,13 +269,8 @@ public function csv_from_result($query, $delim = ',', $newline = "\n", $enclosur
274269
* @param array $params Any preferences
275270
* @return string
276271
*/
277-
public function xml_from_result($query, $params = array())
272+
public function xml_from_result(CI_DB_result $query, $params = array())
278273
{
279-
if ( ! is_object($query) OR ! method_exists($query, 'list_fields'))
280-
{
281-
show_error('You must submit a valid result object');
282-
}
283-
284274
// Set our default values
285275
foreach (array('root' => 'root', 'element' => 'element', 'newline' => "\n", 'tab' => "\t") as $key => $val)
286276
{

0 commit comments

Comments
 (0)