Skip to content
This repository was archived by the owner on Feb 3, 2021. It is now read-only.
This repository was archived by the owner on Feb 3, 2021. It is now read-only.

Price format and json encoded texts? #108

Description

@Piszi

Hello,

I was going through the code of v. 2.3.4 after we saw wrong data in GA Conversions and I saw this:
'price' => number_format($product['price'], 2, ',') I belive this is the wrong format for GA because this will return for example this: 1,000.00 and GA iterpret this as 1 instead of 1000. According to Google the correct format would be 1000.00 so without the comma because GA does not respect currency formatting.

Can you please tell why you choose this formating for the price?

And at the same place I don't really understand why are the names json encoded. This will result in unreadable names if you have UTF-8 chars in your names like: í,é,á,ő,ű,ú etc..

Is there a reason for json encodeing?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions