Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.1.15

Bug #13803 switches not properly indenting with IndentStyles(style=bsd)
Submitted: 2008-04-29 21:47 UTC
From: terrafrost Assigned: clbustos
Status: Assigned Package: PHP_Beautifier (version 0.1.13)
PHP Version: Irrelevant OS:
Roadmaps: 1.0.0RC1    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 33 - 21 = ?

 
 [2008-04-29 21:47 UTC] terrafrost (Jim Wigginton)
Description: ------------ Run the following on the test script: php_beautifier --filters "IndentStyles(style=bsd)" source.php source.txt Test script: --------------- <?php $a='b'; switch ($a){case 'a':case 'b':case 'c':break;default:break;} ?> Expected result: ---------------- <?php $a = 'b'; switch ($a) { case 'a': case 'b': case 'c': break; default: break; } ?> Actual result: -------------- <?php $a = 'b'; switch ($a) { case 'a': case 'b': case 'c': break; default: break; } ?>

Comments

 [2010-03-16 11:13 UTC] clbustos (Claudio Bustos)
-Assigned To: +Assigned To: clbustos -Roadmap Versions: +Roadmap Versions: 1.0.0RC1
 [2010-08-06 02:51 UTC] jespino (Jesús Espino)
Thats problem is related to another 2 problems, the Default.filter.php remove one indentation level on breaks, that no happen when use Pear filter, and the allman indenting style have a problem with the switch formating, explained in the bug #10019, when we solve that bug, this bug will be solved. I don't mark it as duplicated because i'm not sure the behavior of Default filter is 100% correct.