magento / magento/magento2-phpstorm-plugin

UCT live inspection only includes PHP and XML files

Open
#1,101 4 comments 0 reactions 1 assignee View on GitHub

@bohdan-harniuk is already working on this.

Since May 16, 2022.

bug
Dominant language
Java
Stars
463
Forks
98
Avg merge
1h 6m
Merged PRs (30d)
3

Description

**Describe the bug** (*)
The built-in UCT only checks PHP and XML files in live inspection. For example, checking `magepal/magento2-core` version 1.1.12, bundle UCT returns a bunch of deprecated warnings for `escapeHtml()` in `view/adminhtml/templates/system/config/field/extensions.phtml` which are not reflected in live inspection.

**To Reproduce** (*)

Steps to reproduce the behaviour:
1. Install `magepal/magento2-googletagmanager` 2.7.1 in a Magento instance using composer (`magepal/magento2-core` is installed also as a requirement): `composer require magepal/magento2-googletagmanager`
2. Analyse `magepal/magento2-core` from the `vendor`folder using the built-in UCT
3. After UCT finishes its analysis, open `view/adminhtml/templates/system/config/field/extensions.phtml` from the analysis results
4. Live inspection of file above should highlight the errors found by UCT

**Expected behaviour** (*)

Issues in PHTML and other file extensions supported by UCT should be highlighted in live inspection.

**Screenshots**

Screenshot 2022-05-16 at 11 47 55

**Please complete the following information:** (*)

- OS: Mac OS 12.1 Montereey
- PhpStorm/Intellij version: 2022.1
- Plugin Version: 4.3.1

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.