[evolution/webkit-composer: 184/231] Makes links checking UTF-8 aware.
- From: Tomas Popela <tpopela src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution/webkit-composer: 184/231] Makes links checking UTF-8 aware.
- Date: Fri, 13 Sep 2013 14:21:26 +0000 (UTC)
commit 36e588a918eca787541a0809bbf99a69f04c0db7
Author: Tomas Popela <tpopela redhat com>
Date: Wed Apr 17 14:43:00 2013 +0200
Makes links checking UTF-8 aware.
e-util/e-editor-widget.c | 25 ++++++++++++++++++-------
1 files changed, 18 insertions(+), 7 deletions(-)
---
diff --git a/e-util/e-editor-widget.c b/e-util/e-editor-widget.c
index a516d67..2154874 100644
--- a/e-util/e-editor-widget.c
+++ b/e-util/e-editor-widget.c
@@ -310,7 +310,7 @@ editor_widget_check_magic_links (EEditorWidget *widget,
gchar **urls;
GRegex *regex = NULL;
GMatchInfo *match_info;
- gint start_pos_url, end_pos_url, end_input;
+ gint start_pos_url, end_pos_url;
WebKitDOMNode *node;
const gchar *url_pattern = "((([A-Za-z]{3,9}:(?:\\/\\/)?)(?:[\\-;:&=\\+\\$,\\w]+@)?"
@@ -324,11 +324,9 @@ editor_widget_check_magic_links (EEditorWidget *widget,
return;
node_text = webkit_dom_text_get_whole_text (WEBKIT_DOM_TEXT (node));
- if (!node_text)
+ if (!node_text || !g_utf8_validate (node_text, -1, NULL))
return;
- end_input = webkit_dom_range_get_end_offset (range, NULL) - 1;
-
regex = g_regex_new (url_pattern, 0, 0, NULL);
if (!regex) {
@@ -341,13 +339,24 @@ editor_widget_check_magic_links (EEditorWidget *widget,
if (urls) {
gchar *html, *url, *final_url;
+ gchar *url_start_raw, *url_end_raw;
+ glong url_start, url_end;
WebKitDOMDocument *document;
WebKitDOMDOMWindow *window;
WebKitDOMDOMSelection *selection;
g_match_info_fetch_pos (match_info, 0, &start_pos_url, &end_pos_url);
- url = g_strndup (urls[0], end_input - start_pos_url);
+ /* Get start and end position of url in node's text because positions
+ * that we get from g_match_info_fetch_pos are not UTF-8 aware */
+ url_start_raw = g_strndup(node_text, start_pos_url);
+ url_start = g_utf8_strlen (url_start_raw, -1);
+
+ url_end_raw = g_strndup(node_text, end_pos_url);
+ url_end = g_utf8_strlen (url_end_raw, -1);
+
+ /* Remove space on end */
+ url = g_utf8_substring (urls[0], 0, g_utf8_strlen (urls[0], -1) - 1);
/* Select the link and put it inside <A> */
document = webkit_web_view_get_dom_document (WEBKIT_WEB_VIEW (widget));
@@ -356,8 +365,8 @@ editor_widget_check_magic_links (EEditorWidget *widget,
webkit_dom_dom_selection_set_base_and_extent (
selection, webkit_dom_range_get_end_container (range, NULL),
- end_input, webkit_dom_range_get_end_container (range, NULL),
- start_pos_url, NULL);
+ url_end - 1, webkit_dom_range_get_end_container (range, NULL),
+ url_start, NULL);
if (g_str_has_prefix (url, "www"))
final_url = g_strconcat ("http://", url, NULL);
@@ -370,6 +379,8 @@ editor_widget_check_magic_links (EEditorWidget *widget,
webkit_dom_dom_selection_modify (selection, "move", "right", "character");
+ g_free (url_start_raw);
+ g_free (url_end_raw);
g_free (html);
g_free (url);
g_free (final_url);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]