Improve context detection around interfaces in class declarations

Made GetCurrentInstruction return the class declaration as a whole even
if theres a `,` in it (list of implemented interfaces)
Made the only completion for the second word after an extended class to be
`implements`

autoload/phpcomplete.vim

@@ -274,6 +274,8 @@ function! phpcomplete#CompletePHP(findstart, base) " {{{
 		" }}}
 	elseif context =~? 'implements'
 		return phpcomplete#CompleteClassName(a:base, ['i'], current_namespace, imports)
+	elseif context =~? 'extends\s\+.\+$'
+		return ['implements']
 	elseif context =~? 'extends'
 		let kinds = context =~? 'class\s' ? ['c'] : ['i']
 		return phpcomplete#CompleteClassName(a:base, kinds, current_namespace, imports)
@@ -1208,7 +1210,7 @@ function! phpcomplete#GetCurrentInstruction(line_number, col_number, phpbegin) "
 				\ '!', '@', '%', '^', '&',
 				\ '*', '/', '-', '+', '=',
 				\ ':', '>', '<', '.', '?',
-				\ ';', '(',  '|', '['
+				\ ';', '(', '|', '['
 				\ ]
 
 	let phpbegin_length = len(matchstr(getline(a:phpbegin[0]), '\zs<?\(php\)\?\ze'))
@@ -1317,8 +1319,8 @@ function! phpcomplete#GetCurrentInstruction(line_number, col_number, phpbegin) "
 
 	" there were a "naked" coma in the instruction
 	if first_coma_break_pos != -1
-		if instruction !~? '^use' " use ... statements should not be broken up
-			let pos = -1 * first_coma_break_pos + 1
+		if instruction !~? '^use' && instruction !~? '^class' " use ... statements and class delcarations should not be broken up by comas
+			let pos = (-1 * first_coma_break_pos) + 1
 			let instruction = instruction[pos :]
 		endif
 	endif

tests/CompleteClassName_test.vim

@@ -125,7 +125,7 @@ fun! TestCase_complete_classes_from_built_in_classes()
     \   'name': 'FindMeFoo'
     \ },
     \}
-    " the completion should give the value of the 'name' property regardeless
+    " the completion should give the value of the 'name' property regardless
     " of the outer dictionary keys
     let res = phpcomplete#CompleteClassName('tra', ['i'], '\', {})
     call VUAssertEquals([

tests/GetCurrentInstruction_test.vim

@@ -139,5 +139,11 @@ fun! TestCase_returns_instuction_string()
     call VUAssertEquals(
                 \ '$foo->',
                 \ res)
+    call cursor(96, 48)
+    let res = phpcomplete#GetCurrentInstruction(96, 48, [1, 1])
+    call VUAssertEquals(
+                \ 'class Foo extends Bar implements ArrayAccess, It',
+                \ res)
+
     silent! bw! %
 endf

tests/fixtures/GetCurrentInstruction/instructions.php

@@ -92,3 +92,5 @@ public function bar()
 ;
 return $foo-> // cursor here
 ;
+
+class Foo extends Bar implements ArrayAccess, It