notice: Undefined index: min in [...]/formatted_number.module on line 389 [#466572]

I get the following error from the module:

notice: Undefined index: min in [...]/sites/all/modules/formatted_number/formatted_number.module on line 389.

I haven't had time to explore it further, but I will probably do so eventually. Until then, here are some system info:
Drupal 6.x (CVS), PostgreSQL 8.3.7, PHP "5.2.9-pl2-gentoo with Suhosin-Patch 0.9.7", Linux 2.6.28-gentoo-r5

Comments

Comment #1

markus_petrux CreditAttribution: markus_petrux commented 19 May 2009 at 08:22

Status:

Active

» Postponed (maintainer needs more info)

It looks like the field does not have the information that should have been saved when the field settings panel was sent.

Could you please edit the field settings, save, and then test again if the problem persists?

Otherwise, please try to describe where do you exactly get the error? Steps to reproduce?

Comment #2

Freso CreditAttribution: Freso commented 19 May 2009 at 16:38

Project:	Formatted Number	» Money field
Version:	6.x-1.1	» 6.x-1.x-dev
Status:	Postponed (maintainer needs more info)	» Active

I actually think this is a bug with the Money CCK field module, as that is the field I'm using - which in turn uses the Formatted Number CCK module.

Steps to reproduce: Make a Drupal (6) site, install the Money CCK field module (and disable all other, non-needed, modules, if you want), make a new content type and add a Money field. After having configured the field, the error should start popping up.

Comment #3

markus_petrux CreditAttribution: markus_petrux commented 20 May 2009 at 07:04

Project:	Money field	» Formatted Number
Status:	Active	» Needs review

Oh, I see. Thanks for the explanation. :)

Well, I'm transfering back the issue to the Formatted Number CCK module, as the easiest fix would be here.

Could you please try the following mini-patch to formatted_number.module?

   $field_decimals = (isset($field['decimals']) && (int)$field['decimals'] >= 0 ? (int)$field['decimals'] : 0);
-  $field_min = parse_formatted_number($field['min']);
+  $field_min = (isset($field['min']) ? parse_formatted_number($field['min']) : NULL);
   if (!is_numeric($field_min)) {

This should fix the PHP notice caused by the fact that Money CCK field reuses the Formatted Number widget without implementing the min/max options.