sharing a url is not displayed correctly in message view

Bug #1367802 reported by Bill Filler
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Telegram app
Fix Released
Critical
Penk Chen
messaging-app
Fix Released
Critical
Tiago Salem Herrmann
messaging-app (Ubuntu)
Fix Released
Critical
Tiago Salem Herrmann

Bug Description

build 23 on Krillin
- open browser
- go to mobile.nytimes.com
- click one of the news stories displayed
- after page loads, press the "Share" button from the Action menu
- Select messaging-app
- Note that the url looks correct while it's in the entry field
- send the message

Expected result:
- the shared url should be shown correctly in the messaging thread

Actual result:
- the display of the url is all messed up, see the attached picture

The url that is trying to be shared is:
http://mobile.nytimes.com/2014/09/10/us/obama-isis-congress.html

Related branches

Revision history for this message
Bill Filler (bfiller) wrote :
Changed in messaging-app:
importance: Undecided → High
assignee: nobody → Tiago Salem Herrmann (tiagosh)
tags: added: rtm14 touch-2014-09-25
Revision history for this message
Bill Filler (bfiller) wrote :

the issue is easy to reproduce if you simply send a message to the ubuntu phone with this url:
http://mobile.nytimes.com/2014/09/10/us/obama-isis-congress.html

It won't be shared correctly

Bill Filler (bfiller)
Changed in messaging-app (Ubuntu):
importance: Undecided → High
Changed in messaging-app:
status: New → In Progress
Changed in messaging-app (Ubuntu):
status: New → In Progress
Changed in libqtelegram:
importance: Undecided → Critical
Changed in messaging-app (Ubuntu):
importance: High → Critical
Michał Karnicki (karni)
Changed in libqtelegram:
status: New → Confirmed
Michał Karnicki (karni)
Changed in libqtelegram:
assignee: nobody → Michał Karnicki (karni)
Bill Filler (bfiller)
Changed in messaging-app (Ubuntu):
assignee: nobody → Tiago Salem Herrmann (tiagosh)
Changed in messaging-app:
importance: High → Critical
Revision history for this message
Dennis O'Flaherty (doflah) wrote :

I had already filed this as bug #1363292 and implemented a fix on a branch along with some unit tests. My fix allows you to send both phone numbers and links in the same message, so it should be more robust. Do you want to take a look at that instead?

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package messaging-app - 0.1+14.10.20140919-0ubuntu1

---------------
messaging-app (0.1+14.10.20140919-0ubuntu1) utopic; urgency=low

  [ Gustavo Pichorim Boiko ]
  * Increase the time waiting for the OSD to hide.

  [ Tiago Salem Herrmann ]
  * Don't try to linkify phone numbers if web links are present. (LP:
    #1367802)
  * Fix auto-scrolling on the MultiRecipientInput widget. (LP: #1356811)
 -- Ubuntu daily release <email address hidden> Fri, 19 Sep 2014 21:58:41 +0000

Changed in messaging-app (Ubuntu):
status: In Progress → Fix Released
Michał Karnicki (karni)
Changed in libqtelegram:
assignee: Michał Karnicki (karni) → Penk Chen (penk)
Bill Filler (bfiller)
Changed in messaging-app:
status: In Progress → Fix Released
Penk Chen (penk)
Changed in libqtelegram:
status: Confirmed → In Progress
Revision history for this message
Penk Chen (penk) wrote :
Changed in libqtelegram:
status: In Progress → Fix Committed
Michał Karnicki (karni)
Changed in libqtelegram:
milestone: none → m4-beta1
Michał Karnicki (karni)
Changed in libqtelegram:
milestone: m4-beta1 → m3-alpha3
Michał Karnicki (karni)
Changed in libqtelegram:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.