Eclipse Community Forums: PHP Development Tools (PDT)

Help

Home

Home » Language IDEs » PHP Development Tools (PDT) » Formatter Nullable(Formatter remove Nullable)

Show: Today's Messages :: Show Polls :: Message Navigator

Formatter Nullable [message #1749528]

Wed, 07 December 2016 18:49

Rene Dziuba

Messages: 4
Registered: December 2016

Junior Member

Hey guys,

I hope anyone read this posts.
I can not post a bug on github. I can not post a bug on https://bugs.eclipse.org/bugs/enter_bug.cgi?product=pdt

I create a class with one method. The method has PHP 7.1 nullable return type.

PHP Class before formatter:

<?php
class Test
{
    public function get(string $name, int $index): ?array
    { 
    }
}

When I format this class (CONTROL + STRG + F) remove the question mark.

<?php
class Test
{
    public function get(string $name, int $index): array
    { 
    }
}

I found on github this lines in:
File: org.eclipse.php.formatter.core.CodeFormatterVisitor
Method: visit()
Line start: 3409
[code]
...

if (functionDeclaration.getReturnType() != null) {
appendToBuffer(COLON);
insertSpace();
handleChars(lastPosition, functionDeclaration.getReturnType().getStart());
functionDeclaration.getReturnType().accept(this);

lastPosition = functionDeclaration.getReturnType().getEnd();
}

...
[/code>

i can check of PHP 7.1 and with
functionDeclaration.getReturnType().isNullable()

Please update this.

Very thx

Report message to a moderator

Re: Formatter Nullable [message #1749536 is a reply to message #1749528]

Wed, 07 December 2016 21:12

Derek Toolan

Messages: 23
Registered: May 2016

Junior Member

"I can not post a bug on https://bugs.eclipse.org/bugs/enter_bug.cgi?product=pdt":

Can you please send an email to webmaster at eclipse dot org requesting Bugzilla permissions.

Report message to a moderator

Re: Formatter Nullable [message #1749579 is a reply to message #1749536]

Thu, 08 December 2016 10:12

Rene Dziuba

Messages: 4
Registered: December 2016

Junior Member

Thx for answer.

I have send this the webmaster.
PHP 7.1 is since 1 week out.

Why is PHP 7.1 not supported?
I see the changes on github before 4 months ago.

Report message to a moderator

Re: Formatter Nullable [message #1749590 is a reply to message #1749579]

Thu, 08 December 2016 13:08

Michal Niewrzal

Messages: 50
Registered: July 2009

Member

Quote:

Why is PHP 7.1 not supported?

What do you mean exactly? You can create project that will support PHP 7.1 syntax. Of course you can face bigger or smaller problems (like described problem with formatter) but in general PHP 7.1 is supported.

Report message to a moderator

Re: Formatter Nullable [message #1749604 is a reply to message #1749590]

Thu, 08 December 2016 14:15

Rene Dziuba

Messages: 4
Registered: December 2016

Junior Member

I can read under the preferences menu
PHP -> Interpreters -> PHP 7.1 (EXPERIMENTAL)

Oops, pdt has not features for create classes, interfaces and traits.
This is a plugin from others. Embarrassed

Report message to a moderator

Re: Formatter Nullable [message #1749729 is a reply to message #1749604]

Sat, 10 December 2016 11:36

Rene Dziuba

Messages: 4
Registered: December 2016

Junior Member

The formattter works fine with the last-nightly-build.

Very thx for our hard work and a nice weekend.

LG Rene

Report message to a moderator

Re: Formatter Nullable [message #1749738 is a reply to message #1749729]

Sat, 10 December 2016 15:27

Michal Niewrzal

Messages: 50
Registered: July 2009

Member

Nice to hear that Smile

At the end of December there will be official release of PDT (Eclipse Neon.2). Can you close bugzilla issue with annotation that it works ok with your build?

Report message to a moderator

Previous Topic:	Turning PDT to a language server API?
Next Topic:	Support for HHVM (specifically remote debugging with Xdebug)

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Fri Apr 19 23:16:20 GMT 2024

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter