JSON.parse syntax error

[Tersoal]

Hi all.

I've a syntax error in the options parsing in twig:

fields.html.twig line 54: {{ options | json_encode() | raw }}

I think that uploadConfig object isn't properly parse, because the slash in the routes are not escaped (they are '' instead of '').

Is this possible?

[Tersoal]

I use windows, y don't know if I've problems with routes, because windows uses \ instead of /

[Tersoal]

Yes, the problem is the \ generate by windows instead of /.

I've resolved it replacing the urls \ in CroppableImageType, before line 139:

    foreach($options['uploadConfig'] as $key => $optiontotransform) {
        $options['uploadConfig'][$key] = str_replace('\\', '/', $optiontotransform);
    }

It would be no problem in linux machines, I hope. And this will resolve the problem in all environments.

[comur]

Hi,

I've never tested in Windows environment so thanks for your tip. Can you
create a PR so others can also use your workaround please ?

Thanks

Can OMUR
omur.can@gmail.com
+33 (0)6.79.66.58.28

2014-11-16 0:53 GMT+01:00 Tersoal notifications@github.com:

Yes, the problem is the \ generate by windows instead of /.

I've resolved it replacing the urls \ in CroppableImageType, before line
139:

foreach($options['uploadConfig'] as $key => $optiontotransform) {
    $options['uploadConfig'][$key] = str_replace('\\', '/', $optiontotransform);
}

It would be no problem in linux machines, I hope. And this will resolve
the problem in all environments.

—
Reply to this email directly or view it on GitHub
#20 (comment)
.

[Tersoal]

Sure. I'll try it, due to I'm not skilled in fork projects. And I'll try to fork both branches, because I'm using only the bootstrap 3 branch.

Please, test my updates in unix to be sure that it works in all environments.

Thanks.

[comur]

Ok, but you can just fork master and send me PR on master, i'll merge
master on 1.0 anyway

Can OMUR
omur.can@gmail.com
+33 (0)6.79.66.58.28

2014-11-17 10:00 GMT+01:00 Tersoal notifications@github.com:

Sure. I'll try it, due to I'm not skilled in fork projects. And I'll try
to fork both branches, because I'm using only the bootstrap 3 branch.

Please, test my updates in unix to be sure that it works in all
environments.

Thanks.

—
Reply to this email directly or view it on GitHub
#20 (comment)
.

[Tersoal]

OK, perfect.

I'll review some features that does not works very well, like upload image button. I'll update all that I view wrong.

Cheers.

[Tersoal]

Pull request done.