This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 268556 - Incorrect formatting of nested PHP arrays
Summary: Incorrect formatting of nested PHP arrays
Status: RESOLVED DUPLICATE of bug 268171
Alias: None
Product: php
Classification: Unclassified
Component: Formatting & Indentation (show other bugs)
Version: 8.2
Hardware: All All
: P3 normal with 1 vote (vote)
Assignee: Tomas Mysik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-17 12:13 UTC by josef.sabl
Modified: 2016-10-19 07:45 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
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 josef.sabl 2016-10-17 12:13:12 UTC 
Nested arrays are indented incorrectly when autoformatting PHP files.

Expected:

<?php

$myArray = [
	[
		'hello' => 'world',
	],
	[
		'goodbye' => 'cruel world',
	],
];


Actual:

<?php

$myArray = [
		[
		'hello' => 'world',
	],
		[
		'goodbye' => 'cruel world',
	],
];
Comment 1 Tomas Mysik 2016-10-17 12:21:28 UTC 
IMHO duplicate of issue #268171 which is already fixed in the dev build - could you please verify it? Once verified, it can be part of the patch for NB 8.2.

Thanks!

*** This bug has been marked as a duplicate of bug 268171 ***
Comment 2 josef.sabl 2016-10-17 12:32:54 UTC 
Sorry, I won't install dev just to test this as it looks like exactly the same problem :-) Thank you.
Comment 3 Tomas Mysik 2016-10-17 12:58:31 UTC 
(In reply to josef.sabl from comment #2)
> Sorry, I won't install dev just to test this as it looks like exactly the
> same problem :-) Thank you.

In such a case, you can hardly verify that the issue is already fixed which needs to be done in order to be a part of the patch for NB 8.2.

Thanks.
Comment 4 josef.sabl 2016-10-17 15:42:01 UTC 
Your users need to verify bugfixes prior to their release? Funny :) Anyway, I wanted to do that and donwloaded full NB dev as PHP version is not available. And I found out that I also need to install JDK. I am really sorry about that but I am not gonna do it. Hopefully somebody from the other duplicate will verify. Cheers!
Comment 5 Tomas Mysik 2016-10-18 08:03:40 UTC 
(In reply to josef.sabl from comment #4)
> Your users need to verify bugfixes prior to their release? Funny :)

Not sure why you think so. It is normal that reporter verifies that the issue is fixed, using a dev build. Waiting for the next release is possible but (a) it usually takes a looong time and (b) the issue can still be there since it has not been verified, ideally by reporter ;)

> Anyway,
> I wanted to do that and donwloaded full NB dev as PHP version is not
> available.

Strange, I can see it [1].

Thanks.
[1] http://bits.netbeans.org/dev/nightly/latest/
Comment 6 josef.sabl 2016-10-19 07:45:59 UTC 
I can see it now as well, strange.
http://bits.netbeans.org/download/trunk/nightly/latest/

Somebody already confirmed it though. Anyway, thanks for your time.