Blank lines in address fields

[bent]

We have been using Sugar for two years now and have now upgraded from 5.0 to 5.1 on a Debian Etcch LAMP server. After having done so I have noted the following problem:

Everytime I save the edit views from the leads or contacts module (actually all modules that contain address fields) sugar adds me blank lines to the address_street-fields. That happens everytime I save, having edited the address fields or not.

This is quite annoying because these empty lines keep appearing in the details views, serial letters etc.

Has anyone encountered the same problem or a solution?

Thanks a lot!

[breyton5]

Hello!

I've got the same problem here (with 5.1.0 Build 4732).

How can I fix this problem?

Thanks a lot!

[bent]

Hallo I have not yet encountered the reason for this bevavior nor a solution. However, being two now, may be we can narrow down the problem a bit.

(1)
I did an upgrade from 5.0.0f to 5.1 when it happened? How about you?

(2)
Have you any hypothesis where it may come from?

[russlevy]

I have also had the error. I haven't looked at the code yet, but here is an sql statement that will fix it. I have to say though, it's not very good to have to run an SQL statement just to get your data fixed!

update accounts set shipping_address_street=trim(both '\n' from shipping_address_street),billing_address_street = trim(both '\n' from billing_address_street);

[dlorenzetti]

Hello All,

I think It is a bug of Sugar 5.1,

For address street fields Sugar can have more 3 fields <type>_address_street_2/3/4, so, Sugar call the function "add_address_streets" in SugarBean.php, and if your module have these fields Sugar add a new line and put each address.
But if not exists these fields Sugar put the blank line.
For solution that you can fix it directly in /data/SugarBean.php "funtion add_address_streets()"

for each extra fields (address street 2, 3 and 4) you verify if the field is not empty

  
        if ( isset($this->$street_field_2) && !empty($this->$street_field_2)) {
            $this->$street_field .= "\n". $this->$street_field_2;
        }
 
        if ( isset($this->$street_field_3) && !empty($this->$street_field_3)) {
            $this->$street_field .= "\n". $this->$street_field_3;
        }
 
        if ( isset($this->$street_field_4) && !empty($this->$street_field_4)) {
            $this->$street_field .= "\n". $this->$street_field_4;
        } 
  

I already posted this bug in Sugar Bug.

I hope this helps.

[breyton5]

Hello again!

Thanks for this workaround!

Changing the method with the further condition fixed the bug

[bent]

W onderful!
Thank you - works beautifly again!

If this is a 5.1 bug I am sure it will be fixed with the next upgrade because this error is quite annoying.

[thowden]

Another vote of thanks for this fix.