FAQ
Edit report at http://pear.php.net/bugs/bug.php?id=13803&edit=1

ID: 13803
Updated by: jespinog@gmail.com
Reported By: terrafrost at php dot net
Summary: switches not properly indenting with
IndentStyles(style=bsd)
Status: Assigned
Type: Bug
Package: PHP_Beautifier
Package Version: 0.1.13
PHP Version: Irrelevant
Assigned To: clbustos
Roadmap Versions:
New Comment:

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.


Previous Comments:
------------------------------------------------------------------------

[2010-03-16 07:13:28] clbustos

-Assigned To:
+Assigned To: clbustos
-Roadmap Versions:
+Roadmap Versions: 1.0.0RC1


------------------------------------------------------------------------

[2008-04-29 17:47:02] terrafrost

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;
}
?>

------------------------------------------------------------------------

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedAug 5, '10 at 10:51p
activeAug 5, '10 at 10:51p
posts1
users1
websitepear.php.net

1 user in discussion

Jespinog: 1 post

People

Translate

site design / logo © 2022 Grokbase