fix bug (prev_node instead of node) introduced in my last commit

martinpopel · martinpopel · commit 984a9ea5c1f9 · 2018-04-10T17:55:01.000+02:00
diff --git a/udapi/block/tutorial/addcommas.py b/udapi/block/tutorial/addcommas.py
@@ -10,25 +10,20 @@ class AddCommas(Block):
     """Heuristically insert nodes for missing commas."""
 
     def __init__(self, language='en', **kwargs):
-        """Create the AddCommas block object.
-
-        Args:
-        `language`: which language-specific rules to use ('en' or 'cs')
-        """
         super().__init__(**kwargs)
         self.language = language
 
     def process_node(self, node):
+        # TODO: Your task: implement some heuristics
         if self.should_add_comma_before(node):
             comma = node.create_child(form=',', deprel='punct', upos='PUNCT')
             comma.shift_before_node(node)
 
     def should_add_comma_before(self, node):
-        # TODO: Your task: implement some heuristics
         prev_node = node.prev_node
         if prev_node is None:
             return False
-        if self.language == 'en' and node.lemma == 'however':
+        if self.language == 'en' and prev_node.lemma == 'however':
             return True
         if any(n.deprel == 'appos' for n in prev_node.children):
             return True
