Public Forums 71,145 posts

[doorknob Friend doorknobJoin date: December 2013Posts: 36Downloads:0Uploads:1 Thanks: 2 Thanked: 20 times in 1 posts]

#193201

I tried to use the ‘slash’ syntax with border-radius to produce asymetric corners but I got a divide by zero error in t3/includes/lessphp/less/less.php at line 2018 (v2.0.2). For me this raises two issues:
1 Despite the ‘slash’ syntax variant of border-radius being valid, it is easy to work around by specifying each corner individually with regular syntax (works fine).
2 The php crash could presumably occur randomly whenever a slash is included, whether valid or not. This would make debugging a large less file very difficult. I made the following change to prevent the crash. It didn’t make the border-radius statement compile correctly but I can live with that.

case '/': return $a / $b;
to

case '/': return $b ? ($a / $b) : $a;

Regards
Phil

1 user says Thank You to doorknob for this useful post

1. Arvind Chauhan

[cssyeah Moderator cssyeahJoin date: June 2006Posts: 653Downloads:73Uploads:15 Thanked: 121 times in 81 posts]

#516317

Hi doorknob,

Thank you for bug report, we will re-check the less file and update to T3 Framework.

Thanks again.

[doorknob Friend doorknobJoin date: December 2013Posts: 36Downloads:0Uploads:1 Thanks: 2 Thanked: 20 times in 1 posts]

#519691

The bug is still in v2.1.0 at includes/less/lessphp/less/less.php line 2473.
I changed

case '/': return $a / $b;

to

case '/': return $b == 0 ? 0 : ($a / $b);

This doesn’t report the problem but at least it prevents a crash

[Wall Crasher Developer Wall CrasherJoin date: December 2011Posts: 1113Downloads:0Uploads:15 Thanks: 66 Thanked: 361 times in 300 posts]

#519780

Hi Phil,

Thanks for the information. T3 uses https://github.com/oyejorge/less.php to compile less file.
You can report this problem to the development team here.

We will upgrade this library as soon as the a new release of the library is available.

[Author]

Posts