controllers/file.py: Clearer string comparison

See !45

server/web2py/applications/vidjil/controllers/file.py

@@ -646,14 +646,7 @@ def restart_pre_process():
 
 def match_filetype(filename, extension):
     ext_len = len(extension)
-    if len(filename) < ext_len:
-        return False
-    i = -1
-    while (abs(i) < ext_len):
-        if filename[i] != extension[i]:
-            return False
-        i = i - 1
-    return True
+    return filename[-ext_len:] == extension
 
 def filesystem():
     json = []

[Mathieu Giraud]

Si extension == '', le code n'est pas fonctionnellement équivalent à celui d'avant :-)

[Mikaël Salson]

C'est juste. Quel comportement est souhaitable ?

[Mathieu Giraud]

Mmm... '' est bien un suffixe de n'importe quoi, non ?