Remove filename conventions for do-not-translate in revcheck (#319)

Author: alfsb

scripts/translation/lib/RevcheckFileList.php

@@ -68,7 +68,7 @@ function loadTreeRecurse( $lang , $path )
                 continue;
             }
 
-            if ( RevcheckIgnore::ignore( $key ) )
+            if ( RevcheckIgnore::byName( $key ) )
                 continue;
             $file = new RevcheckFileItem( $key , $entry->getSize() );
             $this->list[ $key ] = $file;

scripts/translation/lib/RevcheckIgnore.php

@@ -13,63 +13,31 @@
  *  +----------------------------------------------------------------------+
  *  | Authors:     André L F S Bacci <ae php.net>                          |
  *  +----------------------------------------------------------------------+
- *  | Description: Files ignored on translation tree.                      |
+ *  | Description: Files ignored on manual tree.                           |
  *  +----------------------------------------------------------------------+
  */
 
 require_once __DIR__ . '/all.php';
 
 class RevcheckIgnore
 {
-    public static function ignore( $filename ) : bool
+    public static function byName( $filename ) : bool
     {
-        // Ignore dot files
-
         if ( $filename[0] == '.' )
             return true;
 
-        // Ignore files other than xml assets
-
         if ( ( str_ends_with( $filename , '.xml' ) || str_ends_with( $filename , '.ent' ) ) == false )
             return true;
 
-        // Ignore autogenerated files
-
-        if ( str_starts_with( $filename , "entities." ) )
-            return true;
-        if ( str_contains( $filename , "/entities." ) )
-            return true;
-        if ( str_contains( $filename , "/versions.xml" ) )
-            return true;
-
-        // Only in English, autogenerated, marked not translatable
-
-        if ( $filename == "contributors.ent" )
-            return true;
-        if ( $filename == "contributors.xml" )
-            return true;
-        if ( $filename == "appendices/license.xml" )
-            return true;
-        if ( $filename == "appendices/extensions.xml" )
-            return true;
-        if ( $filename == "appendices/reserved.constants.xml" )
-            return true;
-        if ( $filename == "reference/datetime/timezones.xml" )
-            return true;
-
-        // Only in translations
-
         if ( $filename == "translation.xml" )
             return true;
 
-        // At least, do not ignore
         return false;
     }
 
-    public static function mark( $filename )
+    public static function byMark( $filename )
     {
         $contents = file_get_contents( $filename );
-        $skip = strpos( $contents , '<?do-not-translate?>' ) !== false;
-        return $skip;
+        return str_contains( $contents , '<?do-not-translate?>' );
     }
 }

scripts/translation/lib/RevcheckRun.php

@@ -82,7 +82,7 @@ private function calculateStatus()
 
             if ( $target == null )
             {
-                if ( RevcheckIgnore::mark( "{$this->sourceDir}/{$source->file}" ) )
+                if ( RevcheckIgnore::byMark( "{$this->sourceDir}/{$source->file}" ) )
                     continue;
 
                 $source->status = RevcheckStatus::Untranslated;