Copyright Check False Positive

Aperta
  • Qualcosa non va?
#481 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
php
Ambito
devtools

Direzione di ricerca

Inizia individuando il controllo Copyright e la logica che emette il relativo WARNING per gli header di style.css del theme. Riproduci il controllo con l’avviso di Copyright di Oenology e i relativi tag License; il lavoro è completato quando questo formato non produce più un WARNING e viene segnalato a livello INFO o superiore.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Theme Check is incorrectly returning a WARNING on the Copyright check, for a Theme with both a GNU-recommended formatting for Copyright notice and also with "License:" and ":License URI:" tags present in style.css.

Here is an example of the Copyright notice that is returning a false positive:

* Oenology WordPress Theme, Copyright (C) 2010-2016 Chip Bennett
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 * 
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 * 
 * Note: see the "Oenology Reference" admin page for copyright and 
 * license information for bundled resources, including fonts and icons

This should return at best INFO, not WARNING, as both "Copyright" and "(C)" - the explicitly searched-for strings - are included. (Understanding that parsing the verbiage could be beyond the ability of the regex here.)

Perhaps a "Copyright:" tag is needed/would resolve the issue?

Lingua principale
PHP
Stelle
372
Fork
114
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di WordPress/theme-check

Tutte le issue di WordPress/theme-check

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.