Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 357189 - Formatter (tab policy) is ignored
Summary: Formatter (tab policy) is ignored
Status: CLOSED INVALID
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: PHP UI CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-09 03:14 EDT by xerxes CLA
Modified: 2020-05-14 11:40 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description xerxes CLA 2011-09-09 03:14:11 EDT
Build Identifier: 20110218-0911

I've set tab policy to 'spaces', but in some cases hitting the 'tab' key makes 'tap' indentation (not 'space').

There's the source:
$thumb = new sfThumbnail(
      500, 500, false, false, 100, 'sfImageMagickAdapter', array(
        'method'  => 'shave_all'
      )
    );
There are only spaces for now. But if you create a new line after 'method' line and press 'Tab', the identation will tab but not spaces.

Reproducible: Always

Steps to Reproduce:
1. Window->Preferences->PHP->Formatter->Tab policy = 'Spaces'
2. There's the source:
$thumb = new sfThumbnail(
      500, 500, false, false, 100, 'sfImageMagickAdapter', array(
        'method'  => 'shave_all'
      )
    );
3. Create a new line after 'method' line and press 'Tab', the identation will be tab but not spaces.
Comment 1 Remy Suen CLA 2011-09-09 07:34:01 EDT
Please open bugs with the PHP editor with the PDT team.

I am going to assume you are using PDT and not PHPEclipse.
Comment 2 xerxes CLA 2011-09-09 12:32:30 EDT
No, I'm using PHPEclipse SR2.
I've tested the same thing in SR1 and the first release. Only the first release doesn't have this bug.
Comment 3 Remy Suen CLA 2011-09-09 13:05:10 EDT
(In reply to comment #2)
> No, I'm using PHPEclipse SR2.

Then you should open a bug with the PHPEclipse people.
http://www.phpeclipse.com/
Comment 4 Zhongwei Zhao CLA 2011-09-10 21:23:59 EDT
not pdt's bug;)
Comment 5 Kalin CLA 2011-10-18 03:50:13 EDT
I close it as invalid following Zhao's comment.