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

markus_petrux’s picture

markus_petrux commented
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?

Freso’s picture

Freso commented
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.

markus_petrux’s picture

markus_petrux commented
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.

markus_petrux’s picture

markus_petrux commented
Status: Needs review » Fixed

Well, the fix is so simple that I went ahead and committed to CVS. It should be available in the next nightly snapshot.

Freso’s picture

Freso commented

Yes, that removed the notice quite nicely. Thank you. :)

Status: Fixed » Closed (fixed)

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