ProperEscapingFunction: flag printf() usages for placeholders being escaped incorrectly
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 42/100
Direção de pesquisa
Comece localizando a regra ProperEscapingFunction e o tratamento que ela faz das chamadas a printf(). Compare os exemplos de PHP reportados e não reportados na issue e, em seguida, encontre a cobertura de testes existente da regra. A tarefa estará concluída quando argumentos de placeholder escapados incorretamente forem reportados, enquanto o escaping correto mostrado e os casos de funções de tradução forem aceitos.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Describe the solution you'd like
When printf() is used, we should ensure that the content in the placeholders are correctly escaped.
What code should be reported as a violation?
printf(
'<a class="%s" href="%s">%s</a>',
esc_url( $class ), // Error.
esc_attr( $url ), // Error.
esc_attr( $content ), // Error.
);
printf(
'<a class="%s" href="%s">%s</a>',
esc_html__( $class, 'domain' ), // Error.
esc_url( $url ),
esc_attr_x( $content, $context, 'domain' ), // Error.
);
What code should not be reported as a violation?
Correct usages of escaping:
printf(
'<a class="%s" href="%s">%s</a>',
esc_attr( $class ),
esc_url( $url ),
esc_html( $content )
);
Correct usages of escaping with translation functions:
printf(
'<a class="%s" href="%s">%s</a>',
esc_attr_x( $class, $context, 'domain' ),
esc_url( $url ),
esc_html__( $content, 'domain' )
);
Additional context
- Linguagem predominante
- PHP
- Estrelas
- 261
- Forks
- 44
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de Automattic/VIP-Coding-Standards
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading itTalvez já em andamento @tomjn assumiu há 6 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifTalvez já em andamento @tomjn assumiu há 6 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 76/100
-
Suppress filters in get_posts false positiveTalvez já em andamento @tomjn assumiu há 7 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 48/100
-
Breaking Change Type: Maintenance
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
Automattic/VIP-Coding-Standards#849 · 1 comentário ·
Todas as issues de Automattic/VIP-Coding-Standards
Issues semelhantes
-
extension/Commercial needs-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 2 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
crazy-goat/rabbit-stream#753 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
opensourcepos/opensourcepos#4743 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
OpenConext/OpenConext-engineblock#2129 ·
Mantenedores costumam responder em até 3 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
pluginsGLPI/datainjection#683 · 1 comentário ·
Mantenedores costumam responder em até 1 dia