Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LS for default PHP stacks not working #16196

Closed
2 of 9 tasks
Katka92 opened this issue Mar 2, 2020 · 6 comments
Closed
2 of 9 tasks

LS for default PHP stacks not working #16196

Katka92 opened this issue Mar 2, 2020 · 6 comments
Labels
area/languages Issues related to Language extensions or plugins integration. kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@Katka92
Copy link
Contributor

Katka92 commented Mar 2, 2020

Describe the bug

PHP default stacks don't have any Language support. Code highlighting, error highlighting, code completion etc.

Che version

  • latest 7.9.0
  • nightly
  • other: please specify

Steps to reproduce

  • Go to Workspace -> Add Worksapce
  • Create and open PHP Symfony workspace
  • Open file symfony-demo-application/src/Controler/BlogController.php
  • No LS features are provided.

Expected behavior

Code should be highlighted according PHP syntax and LS features as error highlighting or code completion should work.

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version) Hosted Che
  • minikube (include output of minikube version and kubectl version)
  • minishift (include output of minishift version and oc version)
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

Screenshot from 2020-03-02 11-51-35

Some browser console logs errors that could be related to the problem:
Screenshot from 2020-03-02 11-48-50

@Katka92 Katka92 added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 2, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Mar 2, 2020
@ibuziuk ibuziuk added severity/P1 Has a major impact to usage or development of the system. area/languages Issues related to Language extensions or plugins integration. team/languages and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Mar 2, 2020
@ibuziuk
Copy link
Member

ibuziuk commented Mar 2, 2020

@tsmaeder could you please take a look?

@JPinkney
Copy link
Contributor

JPinkney commented Mar 2, 2020

I think the problem is that when we switched to using the vscode builtins in che-theia we don't provide a PHP language configuration: https://github.com/eclipse/che-theia/blob/master/generator/src/templates/theiaPlugins.json. Since the PHP filetype is no longer be there the PHP language features won't activate when a PHP file is opened.

When I tested on the 7.10.0-SNAPSHOT and opened up BlogController.php the default language configuration is set to plain text (since PHP is missing) so there also won't be syntax colourization

Related issues:
eclipse-theia/theia#6672
#16114

@amisevsk
Copy link
Contributor

amisevsk commented Mar 2, 2020

Can this be worked around by manually opening a file and setting its type to php? Does the PHP plugin depend on vscode/theia for initialization only or also for finding files?

@JPinkney
Copy link
Contributor

JPinkney commented Mar 2, 2020

Can this be worked around by manually opening a file and setting its type to php?

Unfortunately not, since we removed the @theia/textmate-grammers package the PHP filetype that is associated with the syntax colourization etc doesn't get added to Theia and since we don't have the vscode-builtin-php we technically don't have a "real" PHP language configuration. For some reason there is a php filetype that is available but I believe it's only coming from files extension type of BlogController.php

Does the PHP plugin depend on vscode/theia for initialization only

I think it has to be activated by vscode/theia since activation events is defined in the vscode extension

@tsmaeder
Copy link
Contributor

tsmaeder commented Mar 3, 2020

There is a PR here: eclipse-che/che-theia#648 but Github doesn't autolink it.

@nickboldt nickboldt mentioned this issue Mar 4, 2020
27 tasks
@JPinkney
Copy link
Contributor

JPinkney commented Mar 5, 2020

Closing as eclipse-che/che-theia#648 has now been merged

@JPinkney JPinkney closed this as completed Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/languages Issues related to Language extensions or plugins integration. kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

6 participants