Topic: FA 2.4 Dashboard bug.

Hello,

i have upgraded my FA installation to the latest 2.4 build.
all seems to be working and i like it.. but i got one problem..
when i use the Dashboard "The built in not the extension" i have no text over where there is pie or charts ..
i am running it locally using php 5.6.

please check image for better understanding.
https://ibb.co/jcA8qk

Regards.

Re: FA 2.4 Dashboard bug.

Hello Alaa,

I see you are using the Arabic language and this is good test for me.

A couple of years ago I helped another guy with Arabic language to get a suitable font to be used in the Graphics.
We found some, but I had to fix the leadings and other things with the fonts. I did that and we got some usable fonts.

In the config.php file, there is a variable called
    $UTF8_fontfile    = "FreeSans.ttf";
This fontfile doesn't work with the Arabic language. I have listed some utf8 fonts that can be used with the Arabic language.

Will you help and test them and give me a high five (1 - 5) character of the usability of the fonts?

Remember to spell the font names exactly. The smaller the file is the better, but it should be usable of course.

I will then incorporate one of the fonts in the default package, and change the variable in the config.default.php.

/Joe

Post's attachments

ae_AlArabiya.ttf 121.5 kb, 5 downloads since 2017-05-12 

ae_AlManzomah.ttf 118.1 kb, 5 downloads since 2017-05-12 

ae_Tholoth.ttf 177.6 kb, 5 downloads since 2017-05-12 

DejaVuSans.ttf 597.2 kb, 7 downloads since 2017-05-12 

zarnormal.ttf 78.2 kb, 5 downloads since 2017-05-12 

You don't have the permssions to download the attachments of this post.

Re: FA 2.4 Dashboard bug.

Hello joe,
I am more than glad to help with this matter,,

Zarmormal and DejaVuSans are the best for both usability and readability.
so if you looking for the best support and smaller size Zarnormal is great but the font-size needs to be increased by 1-2px. and the words are too close to each other.. i don't mean the letter-spacing, just the space between 2 words.

i'd like to add that DejaVuSerif.ttf doesnt seem to support arabic at all.

Re: FA 2.4 Dashboard bug.

Please check the image.
https://ibb.co/g0djgQ

Re: FA 2.4 Dashboard bug.

Hello again Alaa,

In the file
/reporting/includes/class.graphics.inc

I have increased the font size with 2 pixels. And the space between works has been replaced by two spaces .

Was the font too big or is it ok? And was the space between words better now.

The php GD library doesn't support RTL languages, so we had to fix the strings to do so, but we could not change the graphics itself.

I guess this is the best we can do for now. Maybe there are some skilled developers that can help us in the future. This can always be worked on between releases. It doesn't require any database update.

Please replace the file /reporting/includes/class.graphics.inc with this one.

/Joe

Post's attachments

class.graphic.inc 51.8 kb, 10 downloads since 2017-05-12 

You don't have the permssions to download the attachments of this post.

Re: FA 2.4 Dashboard bug.

Hey joe,,
Thanks alot..
all seems to be much better now..
font size and spacing between words is just perfect
specially when using DejaVuSans.ttf

Respect <3

Re: FA 2.4 Dashboard bug.

Hello...
i started to get this error when viewing dashboard..

Invalid argument supplied for foreach() in file: /home/tajrebat/public_html/FA24/includes/current_user.inc at line 211

/home/tajrebat/public_html/FA24/includes/dashboard.inc:18:     (current_user Object)->check_application_access('GL')
/home/tajrebat/public_html/FA24/admin/dashboard.php:28:     dashboard('GL')

Re: FA 2.4 Dashboard bug.

Hello,

I cannot reproduce this error. Can you tell me a little more detailed about it. I wonder if this is Theme based. Which theme are you using?

/Joe

Re: FA 2.4 Dashboard bug.

It may be your PHP version.
Replace the line 211:

        foreach ($waapp->modules as $module)

with

        $wapp_modules = $waapp->modules;
        foreach ($waapp_modules as $module)

Re: FA 2.4 Dashboard bug.

OK..
i finally got it.
its a weird bug that you can produce by editing the config.php file
$hide_inaccessible_menu_items = 0; to 1.

after doing it dashboard w'll no longer work.
and the following error w'll show up.
Invalid argument supplied for foreach() in file: /home/absys/public_html/Website/includes/current_user.inc at line 211

Kind Regards.

Re: FA 2.4 Dashboard bug.

I will have a look. Maybe we can test for that the menu is set.

Joe

Re: FA 2.4 Dashboard bug.

Yes, there was a bug. And it is fixed and committed to 2.4 unstable. You can download it here and replace in /includes/dashboard.inc.

/Joe

Post's attachments

dashboard.inc 18.8 kb, 13 downloads since 2017-05-21 

You don't have the permssions to download the attachments of this post.

Re: FA 2.4 Dashboard bug.

Thanks a lot...
Confirmed.. its now working perfectly.

Regards.

Re: FA 2.4 Dashboard bug.

Whilst the file attached to the 12th post here pertains to the commit in both branches (v2.3 and v2.4), it has been enhanced in the current master (v2.4.1+) in this commit.