Decimal / currency numbers separator

Login with username, password and session length

[Alejandro]

I am afraid I am not familiar with either of those, so I am glad you figured that out! What are you using them for and why would they care about a print_r function in a theme file which renders as html?

[Julio Sene França]

The problem, probably, occurs because some jquery interaction is being blocked.

I skirted the issue in the Varnish adding the following code to the file default.vcl

Code:

sub vcl_pass {
        # Do not cache these paths.
        if (req.url ~ "^/administrator/.*$" ||
        req.url ~ "^/jreviews/listings/edit/.*$" ||
        req.url ~ "^.*/ajax/.*$" {
        return (pass);
        }
}

[Julio Sene França]

 
My solution is not working 100%. The system shows the same page with or without proxy, but the proxy (I don't know why) blocks the custom fields. The custom fields are in the page, but not showed. Maybe the problem is the script that formats and renders the custom fields in the page. What is this script?

[Alejandro]

Fields data is loaded via ajax using the fields_controller.php and processed on the browser using jreviews.fields.js

[Julio Sene França]

After many tests I finaly found the problem with reverse proxy.

The problem is some kind of warning from fields_controller.php when the value $selected_values[$curr_form_field['name']][0] are empty and the code execute number_format function. This "warning" is ignored by browsers, but the proxy blocks the script when found it. So, the solution is simple: test if the variable are empty before execute number_format.

The final code for fields_controller.php is:

Code:

                   case 'decimal':
                        if($decimal_separator == ',' && !empty($selected_values[$curr_form_field['name']][0])){
                                $fields[$curr_form_field['name']]['selected'] = number_format($selected_values[$curr_form_field['name']][0], 2, ',' ,'.');
//                                $fields[$curr_form_field['name']]['selected'] = $selected_values[$curr_form_field['name']];
                        }
                        else
                                $fields[$curr_form_field['name']]['selected'] = $selected_values[$curr_form_field['name']];
                    break;
                    default:
                        $fields[$curr_form_field['name']]['selected'] = $selected_values[$curr_form_field['name']];
                    break;

[Julio Sene França]

This solution is creating a problem with geomaps because lat/long needs to be in english format.

How can I detect if "Currency Format" property of decimal numbers is set to "yes" in fields_controller.php?

[Alejandro]

The properties are inside this array $curr_form_field['options']. You need to do a print_r be able to see all the array keys.

[Julio Sene França]

The properties are inside this array $curr_form_field['options']. You need to do a print_r be able to see all the array keys.

when I try to put print_r($curr_form_field['options']); into fields_controller.php the script craches and I can't see anything.

If I can detect Currency Format = yes I can apply the conversion just if this option are true.

[Alejandro]

Look in the ajax response using Firebug. I think the setting is using the key "curr_format"

[Julio Sene França]

50% of problem solved.

The final code for /controllers/fields_controller.php is

                    case 'decimal':
                        if($decimal_separator == ',' && !empty($selected_values[$curr_form_field['name']][0]) && $curr_form_field['options']['curr_format']==true){
                                          $fields[$curr_form_field['name']]['selected'] = number_format($selected_values[$curr_form_field['name']][0], 2, ',' ,'.');
                        }
                        else
                                $fields[$curr_form_field['name']]['selected'] = $selected_values[$curr_form_field['name']];
                    break;


now the problem is /models/field.php

I need to identify the same variable to do the correct change. I tried the change bellow, but isn't working.

                        case 'decimal':
                                $inputValue = Sanitize::getString($data['Field'][$fieldLocation],$fieldName);
                                                        $inputValue = $inputValue == '' ? null : $inputValue;
//                                if($decimal_separator == ',' && $field['Field']['_params']['curr_format']==true){
                                if($decimal_separator == ','){
                                        $inputValue = str_replace(".", "", $inputValue);
                                        $inputValue = str_replace(",", ".", $inputValue);
                                }
                        break;

[Alejandro]

The settings for the field are not needed there and therefore are not available. You are going to have to use the exact field name, $fieldName, as the check to make your changes/

[Julio Sene França]

Well, this isn't an elegant (and trustable) solution.
Let's change the way to solve this problem.
How can I change the function that add the lat long coordenates to put values using coma and not poit?  Where is the file to change geomaps.mapPopupSimple()?

[Alejandro]

It is in geomaps.js. If I am not mistaken the values come straight from the Google Maps API.