#!/usr/bin/env python from __future__ import absolute_import from __future__ import print_function import optparse import os import sys import subprocess from six.moves import filter from six.moves import map from six.moves import range try: import lister except ImportError as e: print("ImportError: {}".format(e)) print("You need to run the Zulip linters inside a Zulip dev environment.") print("If you are using Vagrant, you can `vagrant ssh` to enter the Vagrant guest.") sys.exit(1) class Record(object): def __init__(self, func): self.depth = 0 self.i = 0 self.line = 1 self.col = 1 self.matcher = func def validate(fn, check_indent=True): text = open(fn).read() def NoStartTag(end_tag): raise Exception(''' No start tag fn: %s end tag: %s line %d, col %d ''' % (fn, end_tag, state.line, state.col)) def start_tag_matcher(s, start_tag): start_line = state.line start_col = state.col state.depth += 1 old_matcher = state.matcher def f(end_tag): problem = None if start_tag != end_tag: problem = 'Mismatched tag.' elif check_indent and state.line > start_line + 1 and state.col != start_col: problem = 'Bad indentation.' if problem: raise Exception(''' fn: %s %s start: %s line %d, col %d end tag: %s line %d, col %d ''' % (fn, problem, s, start_line, start_col, end_tag, state.line, state.col)) state.matcher = old_matcher state.depth -= 1 state.matcher = f state = Record(NoStartTag) def advance(n): for _ in range(n): state.i += 1 if state.i >= 0 and text[state.i - 1] == '\n': state.line += 1 state.col = 1 else: state.col += 1 def looking_at(s): return text[state.i:state.i+len(s)] == s while state.i < len(text): # HTML tags if looking_at("<") and not looking_at("') or tag in ['link', 'meta', '!DOCTYPE']) if not ignore: start_tag_matcher(s, tag) advance(len(s)) continue if looking_at("' or quote_count % 2 != 0): if text[end] == '"': quote_count += 1 end += 1 if end == len(text) or text[end] != '>': raise Exception('Tag missing >') s = text[i:end+1] return s def check_our_files(): parser = optparse.OptionParser() parser.add_option('--modified', '-m', action='store_true', default=False, help='Only check modified files') (options, _) = parser.parse_args() by_lang = lister.list_files( modified_only = options.modified, ftypes=['handlebars', 'html'], group_by_ftype=True) check_handlebar_templates(by_lang['handlebars'], options.modified) check_html_templates(by_lang['html'], options.modified) def check_handlebar_templates(templates, modified_only): # Check all our handlebars templates. templates = [fn for fn in templates if fn.endswith('.handlebars')] if not modified_only: assert len(templates) >= 10 # sanity check that we are actually doing work for fn in templates: validate(fn, check_indent=True) def check_html_templates(templates, modified_only): # Our files with .html extensions are usually for Django, but we also # have a few static .html files. # The file base.html has a bit of funny HTML that we can't parse here yet. # # We also have .html files that we vendored from Casper. # The casperjs files use HTML5 (whereas Zulip prefers XHTML), and # there are also cases where Casper deliberately uses invalid HTML, # so we exclude them from our linter. templates = filter( lambda fn: ('base.html' not in fn) and ('casperjs' not in fn), templates) templates = sorted(list(templates)) if not modified_only: assert len(templates) >= 10 # sanity check that we are actually doing work for fn in templates: # Many of our Django templates have strange indentation. The # indentation errors are often harmless, even stylistically # harmless, but they tend to be in files that might be old # and might eventually require more scrutiny for things like # localization. See github #1236. bad_files = [ 'static/html/5xx.html', 'templates/500.html', 'templates/confirmation/confirm.html', 'templates/corporate/mit.html', 'templates/corporate/privacy.html', 'templates/corporate/terms-enterprise.html', 'templates/corporate/terms.html', 'templates/corporate/zephyr-mirror.html', 'templates/corporate/zephyr.html', 'templates/zerver/accounts_home.html', 'templates/zerver/accounts_send_confirm.html', 'templates/zerver/api.html', 'templates/zerver/api_endpoints.html', 'templates/zerver/apps.html', 'templates/zerver/create_realm.html', 'templates/zerver/emails/followup/day1.html', 'templates/zerver/emails/followup/day2.html', 'templates/zerver/features.html', 'templates/zerver/hello.html', 'templates/zerver/home.html', 'templates/zerver/integrations.html', 'templates/zerver/invite_user.html', 'templates/zerver/left-sidebar.html', 'templates/zerver/login.html', 'templates/zerver/markdown_help.html', 'templates/zerver/register.html', 'templates/zerver/right-sidebar.html', 'templates/zerver/search_operators.html', ] validate(fn, check_indent=(fn not in bad_files)) if __name__ == '__main__': check_our_files()