Comments between statement don't parser

[alexander-akait]

Input:

if (true) {
    $a = 1;
}

// Don't parsed :(
if (false) {
    $a = 2;
}

[ichiriac]

It's voluntary because I need to handle else or elseif nodes (check here from the code) :

https://github.com/glayzzle/php-parser/blob/master/src/parser/if.js#L44

I'm wondering if getting the AST as input for your transformation is not the wrong approach. How comments are handled into JS parser ?

For me, the doc nodes should not be present in the AST as long as they do not belongs to the instructions sets. You can read this old issue here for more details : #6

If think for this special case as the comment is not between an if and an else statement I can handle it, but this case will never work :

if(true) {
  // ... ok
} 
// some explanation (missing)
else {
 // ... ok
}

The best approach would be to extract documentation node into another property like comments, like done by babylon or flow in JS :

[ichiriac]

I've introduced the extractAllDocs option :

<?php $a = 1 + /* Hello */ 2;

Becomes :

Program {
  kind: 'program',
  children: [ Assign { kind: 'assign', operator: '=', left: [Object], right: [Object] } ],
  errors: [],
  comments: [ Doc { kind: 'doc', isDoc: false, lines: [Object] } ] }

But in order to be consistent in principle I should put every comment inside this array and strictly avoid to use them ouside this array. It's also related on an old issue : #62

[alexander-akait]

@ichiriac we implement logic as js prettier based on babylon. He have also comments (https://astexplorer.net/#/gist/3202ee068c3e3f3645588357d931216f/14a080841262821fa733893886d8fc99ac711795) and it is good 👍

[ichiriac]

@nevadascout, @rasmusbe, can you please give me your point of view on this major change on the next release ?

In the next release, I can provide a structure more close to babylon parser with an leadingComments array :

[nevadascout]

Would this be a breaking change?

[ichiriac]

Hi @nevadascout, glad to hear you

Yes this will be a breaking change, but it only concerns the doc nodes. I plan stop using then into children lists or bodies, and attach them directly to nodes by using a leadingComments property.

In fact if you are actually ignoring doc nodes there is no problem, and if you use them to document functions or variable, with this approach it will be simpler (but needs you to update a bit your code)

[nevadascout]

Sounds good, thanks!