Search options (Page 13 of 200)

Hello again,

I see that we already have a money_format that has been used for OS Win in /includes/current_user.inc on line 369:

// function money_format doesn't exist in OS Win.
if (!function_exists('money_format'))
{
    function money_format($format, $number) 
    {
        return price_format($number);
    } 
}    

All windows users have been using this routine for years without problems. Maybe we should include the function here suggested by @PaulSipley instead. I see that this function is listed in PHP.NET as an alternative to money_format. There is also another slightly better version we can use.

What do you think?

/Joe

Hello guys,

I am not sure. I like the grayed versions in the core as is. This means that the report exists, but I am not allowed to see it. In my opinion this is not a problem. I know my place in the organization, so I don't mind.

I understand the question, but not sure if we should change this.

/Joe

Willdo. Committed to stable repo.

Joe

This has been fixed and committed to stable repo.

A fixed file can be downloaded on Mantis.

Thanks for  reporting this

Joe

This has now been fixed and committed to stable repo.

The fixed file can be downloaded here.

/Joe

Committed to stable repo.

/Joe

Hello again, @tom.horton.
This is amazing. I tried to paste 60 characters into the description field. Only 50 characters were accepted. I am using Windows 10, Chrome browser, php 7.4.9, FA 2.4.9 and having no problems.
Would you tell us your environment, tom. Would be interesting to know.

/Joe

@kvvaradha

Ideas are welcome. Thanks.

Joe

Fixed and committed to stable repo. Thanks.

/Joe

Hello @notrinos and all interested parties.

This was NOT a bug in rep 704, but in /includes/types.inc, function payment_person_name. The string returned from here was mistakenly a link and that didn't work in reports. Fixed now in function payment_person_name, and rep704.php has been restored. A similar bug was also in rep601, bank statements, also fixed now.

Committed to stable repo.

/Joe

I will ask Janusz to take a look at thisl

/Joe