Tested with 3.5 : not same bug: post a topic in ar-php support

https://sourceforge.net/p/ar-php/discussion/779212/thread/78fdf0d2/#b9dc

Same think with ar-php 3.6

shnoulle: Can you attach a small sample survey as .lsa to reproduce?

Done,

But have already the fix and it's only with debug=2.

:)

The fix with 3.5:
Undefined variable: en_start
/ar-php/Arabic/Glyphs.php(440)

// need more work to fix lines starts by English words
if ($en_start) {

Think it can be replaced by:

if ($en_index) {

I agree that this is a bug in ar-php - the $en_start variable is defined nowhere.
However your proposed fix does not work, sadly.

I test with 3.5 and seems to work good ?

What is the problem ?

Denis

I have the same problem (With graph statictics and debug to 2:
Undefined offset: -1 at /application/third_party/ar-php/Arabic/Glyphs.php(2281)) using version 3.6 ?

No,

With replacing
if ($en_start) {
by
if ($en_index) {

It fix the undefined offset. Not sure it fix ar-php but fix undefined offset.

No news from ar-php dev. And i can not really test because i don't have a lot of arabic test and didn't understand one word in arabic ...
ANd i don't found how to report patch in sf.net now ???

Try to report it here. http://sourceforge.net/p/ar-php/bugs/

If it works for you please implement the change for 2.05 - seems to be a rare bug anyway.

I can not find the link to report a bug :
https://sourceforge.net/p/ar-php/discussion/779212/thread/409e37f0/?limit=25#eda2

and:
https://sourceforge.net/p/ar-php/discussion/779212/thread/78fdf0d2/

If you find the 'post new bug" button it can be great ;)

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13742

Update to 3.6.0
And fix ONLY the php notice

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13744

Make a fork : https://github.com/Shnoulle/Ar-PHP

And contact dev:
https://sourceforge.net/p/ar-php/discussion/779212/thread/dc72b9db/?limit=25#978f
(by email too)

2.05+ Build 140125 released