Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326556 - When formatting an XML file: Splitting attributes to multiple lines behaves incorrectly
Summary: When formatting an XML file: Splitting attributes to multiple lines behaves i...
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.xml (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.3 M5   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-29 11:52 EDT by Olli Kallioinen CLA
Modified: 2011-01-14 13:25 EST (History)
3 users (show)

See Also:

Attachments
Patch (1.04 KB, patch)
2010-09-29 11:58 EDT, Olli Kallioinen CLA 		nsand.dev: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Olli Kallioinen CLA 2010-09-29 11:52:49 EDT 
Build Identifier: 20100617-1415

Enable the "Split multiple attributes each on a new line" option.

Splitting attributes to multiple lines should only put the attribute on a new line if there are multiple attributes in an element.

At least judging from the code this seems to be a bug. It would be nice to have separate options how the formatting should actually work (not only in this case).


Reproducible: Always

Steps to Reproduce:
1. Go to: Window -> Preferences -> XML -> XML Files -> Editor
2. tick "Split multiple attributes each on a new line"
3.
Comment 1 Olli Kallioinen CLA 2010-09-29 11:58:27 EDT 
Created attachment 179863 [details]
Patch

Patch to fix the issue. When counting the number of attributes in an element the index for counting is not incremented.
Comment 2 Ed Merks CLA 2011-01-14 11:57:34 EST 
Is this bugzilla assigned to a real live human being capable of responding to contributions from the community?
Comment 3 Nick Sandonato CLA 2011-01-14 13:25:24 EST 
Code was checked in earlier today, but hadn't resolved it yet.