Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Missing quickfix suggestions #23

Closed
alsar opened this Issue · 10 comments

2 participants

@alsar

I am using the latest dev version of the plugin but i don't get any quickfix suggestions in my PHP classes.
Neither for missing use statements or unimplemented methods.

As shown here: http://p2-dev.pdt-extensions.org/phpfeatures.html it should work out of the box.

@pulse00
Owner

could you provide a minimal reproducable testcase?

@alsar

How do i do that?

@pulse00
Owner

you could provide a zip/tar file of your complete project if that's possible for you.

@pulse00
Owner

if it's not possible for you i can add more verbose debugging output to the validator and you could provide me that output.

@alsar

I can't provide that particular project, but i just started a new Symfony project and it's the same problem.
I changed to a clean workspace and created a new Composer project and it's the same.

Any ideas?

Maybe i can provide you my workspace configuration and the plugin installation details. If that's any help.

@pulse00
Owner

ok, please paste the content of the following files (in the root of your newly created symfony project):

  • .project
  • .buildpath
@pulse00
Owner

i can reproduce it now, i'll get back to you when i found out the reason.

@pulse00 pulse00 closed this issue from a commit
@pulse00 pulse00 fixes #23 ad70921
@pulse00 pulse00 closed this in ad70921
@pulse00
Owner

@alsar problem fixed. thanks for reporting. a new build is running and should be available shortly.

@alsar

What was the problem?
I just now discovered if i only use the Core plugin (without any other plugins from the http://p2-dev.pdt-extensions.org update site) the quickfix suggestions work.

@pulse00
Owner

I was trying to skip sourcefiles which are not included in the buildpath to improve performance, but that check was too restrictive: ad70921#L0R77

The sourcefile needed to be on the exact entry in the buildpath, deeper entries were not validated...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.