I am getting this too with coder 7.x-2.2 version of the sniffs ( using in PHPStorm ).
In my case the link is :

l("<i class='icon icon-industrial-building'></i>", 'node/add/job', array(
    'attributes' => array('title' => 'add job'),
    'html'       => TRUE
  ));

Is it the case that it should be omitted if the link is html=>true ?

Comments

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Project: Drupal Code Sniffer » Coder
Version: 7.x-1.0-beta1 » 7.x-2.x-dev
Component: Code » Code Sniffer

Drupalcs has been merged into coder, moving this issue.

8bitplateau’s picture

8bitplateau commented
Component: Code Sniffer » Coder Sniffer
Issue summary: View changes

I am getting this too with coder 7.x-2.2 version of the sniffs ( using in PHPStorm ).
In my case the link is :

l("<i class='icon icon-industrial-building'></i>", 'node/add/job', array(
    'attributes' => array('title' => 'add job'),
    'html'       => TRUE
  ));

Is it the case that it should be omitted if the link is html=>true ?

  • klausi committed 63459c3 on 8.x-2.x 
    Issue #1720056: do not complain about HTML tags passed to l().

  • klausi committed f369241 on 7.x-2.x 
    Issue #1720056: do not complain about HTML tags passed to l().
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Version: 7.x-2.x-dev » 8.x-2.x-dev
Status: Active » Fixed

Committed a fix to both branches.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.