Notice: A non well formed numeric value encountered

[TullariS]

abs_paging_height() in reportico_report_html.php throws a notice when $height_string has a percent sign ie "4%". This causes HTML reports to fail, since it counts as an error.

Suggest changing $height_string in line 134 to strstr($height_string, "%", true), in order to strip out the percentage sign.

Also, same thing should apply to abs_paging_width(), line 146.

Apologies if there was a more efficient way of suggesting this change. I'm not very familiar with how Git works...yet.

Thank you

[reportico-web]

Ok thanks ill test that and sort that out very soon.. sounds sensible.

Peter

[ramojo]

Hi. Still encountering the error. Any fix?

[shabaz-ejaz]

@reportico-web Any updates on this issue? It seems this issue has been at stand-still since March. I'm still encountering this.

[reportico-web]

Apologies for late action on this. I just made a commit .. i dont think i should ha

[reportico-web]

I just put a fix up in a version 5.4.1 which you should be able to fetch and try.. let me know if it doesnt work

Also you might like to try a completely re-engineered version if you make your composer require ^6.0 of the module which is in beta. It works with PHP 7, am getting 5.6 PHP supported over next few days ( see here Ill give you more info if you fancy trying it :)

[shabaz-ejaz]

@reportico-web Not working:

The requested package reportico/laravel-reportico 5.4.1 exists as reportico/laravel-reportico[4.1.0, 4.2.0, 4.3.0, 5.1.0, 5.2.0, 5.3.0, 5.4.0, 6.0.0, 6.0.3, dev-develop, dev-master, dev-reportico-new-architecture] but these are rejected by your constraint

I changed my composer.json to this: "reportico/laravel-reportico": "~5.4.1",

I cant see 5.4.1 anywhere on your releases. I only see up to 5.4.0.

[reportico-web]

I forgot to update on packagist .. try now ?