Opened 4 years ago

Closed 4 years ago

#1485990 closed Bugs (fixed)

Autocomplete bugs when erasing the comma

Reported by: Technetux Owned by:
Priority: 5 Milestone: 0.3-stable
Component: Client Scripts Version: git-master
Severity: normal Keywords: autocomplete erase comma searching hide
Cc: ziba, ksteinhoff

Description

  1. use the autocomplete feature when composing a message (it will put in the email and add a comma for you)
  2. erase the comma then stop (the searching message wont go away)
  3. keep erasing, it wont go away until you erase the <

get_input_value in autocomplete.inc is breaking the comparison in app.js>ksearch_query_results one could get by this quite simply by passing back to javascript an unaltered version of the search query as show below

Index: program/steps/mail/autocomplete.inc
===================================================================
--- program/steps/mail/autocomplete.inc (revision 2780)
+++ program/steps/mail/autocomplete.inc (working copy)
@@ -41,7 +41,7 @@
   sort($contacts);
 }

-$OUTPUT->command('ksearch_query_results', $contacts, $search);
+$OUTPUT->command('ksearch_query_results', $contacts, $_POST['_search']);
 $OUTPUT->send();

 ?>

Change History (1)

comment:1 Changed 4 years ago by alec

  • Milestone changed from later to 0.3-stable
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [92f522d8].

Note: See TracTickets for help on using tickets.