Bug 28798 - Fehler bei Suche nach '*'
Fehler bei Suche nach '*'
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.1
Assigned To: Jascha Geerds
Dirk Wiesenthal
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-16 15:18 CEST by Alexander Kläser
Modified: 2012-12-12 21:07 CET (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-10-16 15:18:09 CEST
aus Bug 28002, Comment 7:
> Uncaught SyntaxError: Invalid regular expression: /*/: Nothing to repeat
> AppCenterPage.js:588
> Wenn im Suchfilter * eingegeben wird.
Comment 1 Florian Best univentionstaff 2012-10-19 08:45:20 CEST
Wenn schon regex benutzt wird, dann sollte das Suchpattern auch ein bisschen sanitized werden → In anderen Modulen gibt es nur * als joker. Hier könnte eventuell * durch .* ersetzt werden und das pattern generell escaped werden.
Alternativ sollte die Exception bei einem ungültigem Pattern abgefangen werden und ein rotes Warnsymbol am Input-widget angezeigt werden.
Comment 2 Florian Best univentionstaff 2012-10-19 09:05:30 CEST
Da nach Substrings gesucht wird wäre es auch sinnvoll whitespace am Anfang und Ende zu löschen und einfache spaces durch \s+ zu ersetzen.
Comment 3 Jascha Geerds univentionstaff 2012-10-22 15:49:14 CEST
Folgende Punkte wurden angepasst (2.0.48-1):
 * Whitespace am Anfang/Ende des Suchbegriffs werden entfernt
 * Das '*'-Zeichen kann als Wildcard benutzt werden
 * Spaces werden durch \\s ersetzt

Changelog-Eintrag ist vorhanden.
Comment 4 Dirk Wiesenthal univentionstaff 2012-11-12 21:56:47 CET
Funktioniert. VERIFIED
Comment 5 Stefan Gohmann univentionstaff 2012-12-12 21:07:42 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".