busybox11 / NowPlaying-for-Spotify

A Spotify Connect visualizer
https://nowplayi.ng
GNU General Public License v3.0
183 stars 23 forks source link

Issue with arabic #54

Closed victorbnl closed 3 years ago

victorbnl commented 3 years ago

Parse error: syntax error, unexpected token "\", expecting ")" in C:\xampp\htdocs\assets\lang\ar.php on line 5

victorbnl commented 3 years ago

I think there are too much ' here

\'Spotifyاتصل بـ' \'

VoXaN24 commented 3 years ago

Yes but no, like it's the fact it's in Arabic made that. It's the same amout has other... It's maybe for that i hvn't translated this part

VoXaN24 commented 3 years ago

The issue is still present ?

victorbnl commented 3 years ago

Yes

VoXaN24 commented 3 years ago

Crap, i think i just remove the "Connect to spotify" to something else

victorbnl commented 3 years ago

There's no need

VoXaN24 commented 3 years ago

It's that part made the errors for an unknow reason

victorbnl commented 3 years ago

I changed \' Spotifyاتصل بـ' \' to \' Spotifyاتصل بـ \' and it works. If you need two quotes after the "Login to Spotify" because the arabic language is made like that, just escape also the first one : \' Spotifyاتصل بـ \' \'

victorbnl commented 3 years ago

It's that part made the errors for an unknow reason

I don't think you really needed to change the whole sentence just for this problem though, as I described above

VoXaN24 commented 3 years ago

The PB, i don't modificate anything of the number of ' , i think it just didn't like Arabic, the sentence has the same meaning btw

VoXaN24 commented 3 years ago

Oh, Well i modificated back

victorbnl commented 3 years ago

The PB, i don't modificate anything of the number of ' , i think it just didn't like Arabic, the sentence has the same meaning btw

Ok but why does it work for me then? xD

VoXaN24 commented 3 years ago

Idk that a mystery mate

victorbnl commented 3 years ago

Ok xD Now it works anyway