Let trans_default_domain autocomplete put quotes around domain #526; switch single to double quotes and catch both wraps

Author: Haehnchen

src/fr/adrienbrault/idea/symfony2plugin/templating/completion/QuotedInsertHandler.java

@@ -21,13 +21,15 @@ public static QuotedInsertHandler getInstance(){
 
  @Override
  public void handleInsert(InsertionContext context, LookupElement lookupElement) {
- if(ResourceFileInsertHandler.isStringBeforeCaret(context.getEditor(), context, "'")) {
+ if(ResourceFileInsertHandler.isStringBeforeCaret(context.getEditor(), context, "'") ||
+ ResourceFileInsertHandler.isStringBeforeCaret(context.getEditor(), context, "\""))
+ {
  return;
  }
 
  int startOffset = context.getStartOffset();
- context.getDocument().insertString(startOffset, "'");
- context.getDocument().insertString(startOffset + lookupElement.getLookupString().length() + 1, "'");
+ context.getDocument().insertString(startOffset, "\"");
+ context.getDocument().insertString(startOffset + lookupElement.getLookupString().length() + 1, "\"");
 
  // move to end
  context.getEditor().getCaretModel().moveCaretRelatively(1, 0, false, false, true);

tests/fr/adrienbrault/idea/symfony2plugin/tests/templating/translation/TwigTranslationCompletionContributorTest.java

@@ -124,13 +124,19 @@ public void testTransDefaultDomainCompletionShouldQuotedStringValue() {
  assertCompletionResultEquals(
  TwigFileType.INSTANCE,
  "{% trans_default_domain interchang<caret> %}",
- "{% trans_default_domain 'interchange' %}"
+ "{% trans_default_domain \"interchange\" %}"
  );
 
  assertCompletionResultEquals(
  TwigFileType.INSTANCE,
  "{% trans_default_domain 'interchang<caret>' %}",
  "{% trans_default_domain 'interchange' %}"
  );
+
+ assertCompletionResultEquals(
+ TwigFileType.INSTANCE,
+ "{% trans_default_domain \"interchang<caret>\" %}",
+ "{% trans_default_domain \"interchange\" %}"
+ );
  }
 }