Project:Custom Formatters
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

warning: implode() [function.implode]: Invalid arguments passed in /Applications/MAMP/htdocs/drupal6/sites/all/modules/custom_formatters/custom_formatters.admin.inc on line 68.

The code in question is:

<?php
  $form
['field_types'] = array(
   
'#type' => 'textfield',
   
'#title' => t('Field types'),
   
'#autocomplete_path'  => 'js/formatters/autocomplete',
   
'#default_value' => implode(', ', unserialize($formatter['field_types'])),
   
'#description' => t('A comma-separated list of CCK fields. Example: number_integer, number_decimal.'),
   
'#required' => TRUE,
  );
?>

Looks like $formatter is an empty array at the time.

Comments

#1

Title:Error when creating NEW formatter» Warning when creating NEW formatter

Hi lyricnz,

Right you are, unfortunately the dev environment I've been testing on isn't showing warnings, but I will deal with that.
As it's only a warning it can be ignored, but it will be fixed, version 1.1 is practically ready to go, I just want to port it to 7 before I release it.

Cheers,
Deciphered.

#2

Status:active» fixed

Fixed in 6.x-1.1 and 7.x-1.x-dev.

Cheers,
Deciphered.

#3

Status:fixed» closed (fixed)

#4

Version:6.x-1.0» 6.x-1.x-dev
Priority:normal» minor
Status:closed (fixed)» active

Similar error in new dev snapshot. No 2 & 3 arguments (2 separate errors).

Missing argument 2 for _custom_formatters_tokens_list(), called in \sites\all\modules\custom_formatters\custom_formatters.module on line 319 and defined in \sites\all\modules\custom_formatters\custom_formatters.module on line 331.

#5

Hi Alan,

Good catch, can't believe I let that one slip through. Will be fixed in next development build.

Cheers,
Deciphered.

#6

Status:active» fixed

Fixed and committed to DRUPAL-6--1.

#7

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here