Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 410764

Summary: the second embedded link in message posted to delegatedUI is ignored
Product: [ECD] Orion Reporter: John Swanke <jswanke>
Component: ClientAssignee: Project Inbox <orion.client-inbox>
Status: RESOLVED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: grant_gayed, mamacdon, Szymon.Brandys
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
screenshot of links
none
the correct picture
none
Delegated UI link test plugin none

Description John Swanke CLA 2013-06-13 15:58:40 EDT
this is a regression:

I pass a message with 2 of these meta links: [name](http://link.com)

window.parent.postMessage(JSON.stringify(
   {pageService: "orion.page.delegatedUI", 
   source: "com.xxx.xxx", 
   status: {Message: msg}}), "*");

the first instance is successfully converted to a link 
the second instance is not
Comment 1 Grant Gayed CLA 2013-06-13 16:53:06 EDT
Just to confirm, this is in the context of a contributed editor command, not a Shell page command, right?
Comment 2 Szymon Brandys CLA 2013-06-14 08:36:26 EDT
AFAIK "orion.page.delegatedUI" extension is used only in the context of an editor command.
Comment 3 John Swanke CLA 2013-06-14 08:51:46 EDT
Right-- the context is a contributed editor command returning a status with more then one [link](url) formatted links...
Comment 4 Mark Macdonald CLA 2013-06-14 14:49:14 EDT
Created attachment 232395 [details]
screenshot of links

Hi John

I made a small test plugin testcase, but I haven't been able to recreate the problem. I passed a message containing several URLs like this, and they rendered correctly (see attached picture).

> var msg = "[link1](http://www.ibm.com/) filler text [link2](https://www.ibm.com/foo/bar?a=b&c=d)";
> window.parent.postMessage(JSON.stringify({
>    pageService: "orion.page.delegatedUI",
>    source: "test.delegatedui",
>    Status: {
>        Message: msg
>    }
> }), "*"); 

Can you provide the actual URLs that you are passing in? Do they contain any unusual characters? Perhaps one of your URLs is confusing our URL-detecting regex.
Comment 5 Mark Macdonald CLA 2013-06-14 14:51:11 EDT
Created attachment 232396 [details]
the correct picture
Comment 6 Mark Macdonald CLA 2013-06-14 14:51:57 EDT
Created attachment 232397 [details]
Delegated UI link test plugin

FWIW here is the complete plugin code that I tested with.
Comment 7 John Arthorne CLA 2015-05-05 14:45:10 EDT
Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see:

https://dev.eclipse.org/mhonarc/lists/orion-dev/msg03444.html