Details
-
Sub-task
-
Resolution: Fixed
-
Minor
-
None
-
None
-
None
-
November sprint - webdanica, 2017 sprint - webdanica
Description
Fix C2b - Finder ikke telefonnumre
Seed: http://akadstud.dk/
Webdanica: kb-test-webdanica-001.kb.dk
Seed: https://rent-toej.nu
Webdanica: http://kb-test-webdanica-001.kb.dk
Regex forslag
\b(?:([tT][eE]?lf?|[Nn]r|[Tt]elefon|[Mm]obil)[\.|:]?)\s*(\+)?(45)?\s*(\d{2}\s*\d{2}\s*\d{2}\s*\d{2}|\d{4}\s*\d{4}|\d{8})\b
GitHub:DanicaRegexps.java
Forslag
I DanicaRegexps.java tænker jeg at dette skal udskiftes med forslaget regex udtryk Linie 23 public static String danishTlfRegexp = "(?i)\\b(?:tlf\\.?(nr\\.?)?|telefon|mobil)\\s*(\\+45)?\\s*[1-9]\\d(.?\\d{2}){3}\\b"; !! Denne bruges ikke og skal ikke rettes! public static String danishTlfRegexpNoCase = "\\b(?:tlf\\.?(nr\\.?)?|telefon|mobil)\\s*(\\+45)?\\s*[1-9]\\d(.?\\d{2}){3}\\b";
Kode før
public static String danishTlfRegexp = "(?i)\\b(?:tlf\\.?(nr\\.?)?|telefon|mobil)\\s*(\\+45)?\\s*[1-9]\\d(.?\\d{2}){3}\\b";
Kode efter
public static String danishTlfRegexp = "\\b(?:([tT][eE]?lf?|[Nn]r|[Tt]elefon|[Mm]obil)[\\.|:]?)\\s*(\\+)?(45)?\\s*(\\d{2}\\s*\\d{2}\\s*\\d{2}\\s*\\d{2}|\\d{4}\\s*\\d{4}|\\d{8})\\b";