251 Reply by joe

(8 replies, posted in Report Bugs here)

Yes, it is amazing that this bug is first detected now after so many minor releases.

Maybe because the entrance is mostly done by the Sales routines.

Joe

Go to forum: Report Bugs here

252 Reply by joe

(8 replies, posted in Report Bugs here)

This has been fixed and committed to stable repo. Thanks @barbarian for finding this.

A fixed copy can be downloaded and replaced.

/Joe

Go to forum: Report Bugs here

253 Reply by joe

(2 replies, posted in Items and Inventory)

Yes, it is necessary. The Stock Item List is Not showing again. Fixed and committed to stable.

/Joe

Go to forum: Items and Inventory

254 Reply by joe

(3 replies, posted in Report Bugs here)

This has been fixed and committed to stable repo.

Thanks @boxygen.

/Joe

Go to forum: Report Bugs here

255 Reply by joe

(11 replies, posted in Accounts Payable)

Guys,

Janusz is looking at this and will try to get a solution.

Joe

Go to forum: Accounts Payable

256 Reply by joe

(10 replies, posted in Report Bugs here)

Cash flow statement fixed and sent to extension repo.

Will be updated asap.

/Joe

Go to forum: Report Bugs here

257 Reply by joe

(4 replies, posted in Banking and General Ledger)

Guys,

This is fixed and sent to extension repo maintainer, Itronics. Thanks.

/Joe

Go to forum: Banking and General Ledger

258 Reply by joe

(13 replies, posted in Setup)

Guys.

This has been fixed and sent to extension repo maintainer, Itronics.

/Joe

Go to forum: Setup

259 Reply by joe

(10 replies, posted in Report Bugs here)

Hello again,

The fix for money_format has been replaced by price_format and is committed to stable repo.

The php 8 version is progressing. the bugs with crediting an invoice and direct Invoicing is working again both for customers and suppliers. The fix has just been committed by Itronics to stable repo.

Please help testing if you have time and access to php 8. So we can get php 8 ready.

/Joe

Go to forum: Report Bugs here

260 Reply by joe

(13 replies, posted in Setup)

Hello,

What I mean is that I don't have resources to fix and update all the COAs in the repo. The repo is maintainted by Itronics and is normally updated when new major releases are done. If anything is wrong with a COA we should try to fix the errors and send them to Itronics. (or me and I will send them to Itronics).

/Joe

Go to forum: Setup

261 Reply by joe

(13 replies, posted in Setup)

I am sorry, I only have access to the 2 en_US files (new and demo). Maybe @apmuthu can help us?

Joe

Go to forum: Setup

262 Reply by joe

(10 replies, posted in Report Bugs here)

Ok, I am also using php 8, $go_debug=2 and not getting this error. I am an old C and C++ programmer and these languages always take care of old versions (backward compatible), so I am getting really weird of this php language.

We will simply change the function in pdf_report.inc.

We still have some other issues in php 8. Working on it. One of them is crediting in Customer Transactions.. Not reading the original invoice.

So please do NOT use php 8 seriously yet.

Joe

Go to forum: Report Bugs here

263 Reply by joe

(10 replies, posted in Report Bugs here)

@kvvaradha

I see no problem here. Do you?

/Joe

Go to forum: Report Bugs here

264 Reply by joe

(10 replies, posted in Report Bugs here)

i have amended the text in current_user.inc, line 369.

We will let it be for a while, for backwards compatibility in modules.

/Joe

Go to forum: Report Bugs here

265 Reply by joe

(10 replies, posted in Report Bugs here)

Will have a look.

Joe

Go to forum: Report Bugs here

266 Reply by joe

(12 replies, posted in Setup)

I am not so sure about this. It is very common to let the username be case insensitive.
It is more important that the password is strong. And, really, this is not a Fort Knox issue.

Joe

Go to forum: Setup

267 Reply by joe

(3 replies, posted in Report Bugs here)

Yes, I guess you are right.
I am out of office this week. This will be fixed when in office again.

Joe

Go to forum: Report Bugs here

268 Reply by joe

(18 replies, posted in Report Bugs here)

This mission is now fixed and committed to stable repo. I hope all of them are taken, but please report any remains.

/Joe

Go to forum: Report Bugs here

269 Reply by joe

(18 replies, posted in Report Bugs here)

After talking to Janusz, we can
change the line:

return $row[0];

to

return is_array($row) ? $row[0] : false;

This fixes the problem. But we don't have to do it all over the places. Only where there are NO aggregates as SUM, COUNT etc. These aggregates will always deliver at least one row with empty values.

I will start fixing the needed places tomorrow.

/Joe

Go to forum: Report Bugs here

270 Reply by joe

(18 replies, posted in Report Bugs here)

ok Thanks @kvvaradha.

I will just have a short discussion with Janusz (itronics) about this mysqli issue, before trying to fix it.

I will fix the qty_cell error.

/Joe

Go to forum: Report Bugs here

271 Reply by joe

(18 replies, posted in Report Bugs here)

Did you also try:

if ($result == false)
    return false;

Joe

Go to forum: Report Bugs here

272 Reply by joe

(4 replies, posted in Misc. Charts of Accounts)

@apmuthu

Good observation with SET commands. I will look for how to do it.
Of course if you have it at hand, please advise.

Joe

Go to forum: Misc. Charts of Accounts

273 Reply by joe

(18 replies, posted in Report Bugs here)

@kvvaradha

Couldn't we just put these two lines after the query result:

if (!$result)
    return false;

Because if there is a $result then db_fetch will always return at least one array.

/Joe

Go to forum: Report Bugs here

274 Reply by joe

(7 replies, posted in Report Bugs here)

This has been committed to stable repo.
A downloaded file can be downloaded here and replaced in /includes/ui/

Joe

Go to forum: Report Bugs here

275 Reply by joe

(18 replies, posted in Report Bugs here)

Ok I will have a look at this.

Joe

Go to forum: Report Bugs here