Add a link in a Broadcast message to open a record

[Nicolas Micoud]

Hi,

I've managed to create a process that adds a broadcast message to some users in order they can have a look on some records.

ATM, is just a "raw" text ; is it possible to add a "link" that will open the record (defined via AD_Note.AD_Table_ID/Record_ID)?

Basically, it will be the same behaviour as right click / window on reports.

Or should I made some changes on the BroadcastMessageWindow and a button there ?

Thanks,

Nicolas

[Heng Sin Low]

add a button or it should be possible to modify the broadcastmessagewindow to looks for certain predefined format and render it as a link (like your smartphone able to recognize email and phone number in your message and render it as a clickable link).

--
You received this message because you are subscribed to the Google Groups "iDempiere" group.
To unsubscribe from this group and stop receiving emails from it, send an email to idempiere+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/idempiere/1c17e291-b540-499c-aba5-9d7b62aec0f0n%40googlegroups.com.

[Nicolas Micoud]

I think I can add the button.

But I like the predefined format option (but I'm not sure to have necessary skills).

It would looks like : "start of text #Link_TableID=132_Record_ID=456# end of text" ?

If so, how to generate the link to when use clicks on it, it adds a tab?

[Heng Sin Low]

the reaction for the link should be just a call to AEnv.zoom method to open as a normal window.

To view this discussion on the web visit https://groups.google.com/d/msgid/idempiere/74d3ccc0-20bf-4dbc-82ea-858a46c8f052n%40googlegroups.com.

[muri...@devcoffee.com.br]

Hi @nicolas,

Inside a process, we create and publish the broadcast message for each record scheduled with:

MBroadcastMessage bdMessage = new MBroadcastMessage(getCtx(), 0, get_TrxName());
bdMessage.setBroadcastType(MBroadcastMessage.BROADCASTTYPE_ImmediatePlusLogin);
bdMessage.setBroadcastFrequency(MBroadcastMessage.BROADCASTFREQUENCY_UntilAcknowledge);
bdMessage.setTarget(MBroadcastMessage.TARGET_User);
bdMessage.setAD_User_ID(u.get_ID());
StringBuilder message = new StringBuilder();
message.append("Hi ").append(u.getName()).append(", <br>")
.append("You have an appointment at ")
.append(scheduledDate).append(" . <br>")
.append("Click here to see more information:<br>")
.append(getUrlZoom(po, uuid));

bdMessage.setBroadcastMessage(message.toString());
bdMessage.saveEx();
BroadcastMsgUtil.publishBroadcastMessage(bdMessage.get_ID(), get_TrxName());



And this is the method that creates the link to the window/record: 

private String getUrlZoom(PO po, String uuid) {

    StringBuilder url = new StringBuilder("");

    url.append("<a href=\"javascript:void(0)\" class=\"rp-href\" onclick=\"window.idempiere.zoomWindow(@#clientInfo_BroadcastComponentId@, '");

    url.append(po.get_KeyColumns()[0]);

    url.append("', '").append(po.get_ID()).append("','").append(uuid).append("')\">");

    url.append(po.get_ValueAsString(MCOBContrato.COLUMNNAME_DocumentNo)).append("</a>");

    return url.toString();

}

[Nicolas Micoud]

Hi,

thanks !

I've managed to show the link, it is "clikeable", but when clicking on it, nothing happens :-/

Not sure, but maybe the missing part is "@#clientInfo_BroadcastComponentId@", which is not references elsewhere ?

Nicolas

[matheus....@devcoffee.com.br]

Hi Nicolas !

At devCoffee, we implemented the event to handle this context in BroadCasMessageWindow, if you think this is useful, i can do a pull request or help you to do this. 


Matheus Marcelino

[Nicolas Micoud]

Hi Matheus,

I think is useful, so +1 if you can submit a PR :)

Thanks

Nicolas

[matheus....@devcoffee.com.br]

Nicolas the code below complements what murilo said: 

In MBroadCastMessage.java i've made this changes:

@Override

public String get_Translation (String columnName) {

    String translation = super.get_Translation(columnName );

    if(translation.indexOf('@') > 0)

    return Env.parseContext(getCtx(), 0, translation, false, false);

    return translation;

}

In BroadcastMessageWindow.java:

lines 121 to 124:

Env.setContext(Env.getCtx(), "#clientInfo_BroadcastComponentId", pnlHead.getUuid());

textMsgContent.setContent(mbMessages.get(0).get_Translation(MBroadcastMessage.COLUMNNAME_BroadcastMessage));

pnlHead.addEventListener(ZoomEvent.EVENT_NAME, this);

htmlDiv.setFocus(true);

lines 256 to 262:

} else if(event.getName().equals(ZoomEvent.EVENT_NAME)) {

    Clients.clearBusy();

    ZoomEvent ze = (ZoomEvent) event;

    if (ze.getData() != null && ze.getData() instanceof MQuery) {

            AEnv.zoom((MQuery) ze.getData());

    }

}

If @hengsin approves the idea,  i'll do a pr to be reviewed.

Matheus Marcelino

[Nicolas Micoud]

Is not working :(

I think issue relates to value of uuid of getUrlZoom.
What should it be ? uuid of the record or uuid of BroadcastMessageWindow ?
In his example, I don't see what is the source.

Thanks

Nicolas

[matheus....@devcoffee.com.br]

uuid of the ad_window to zoom. For Example, if po is instanceof MOrder, we need to use the sales order/purchase order window uuid ;)

Matheus Marcelino

[Nicolas Micoud]

ah yes ! works now !!

thanks

Haven't tested but I think it would be easier to passe the tableID and find the window (on BroadcastMessageWindow.onEvent) using Env.getZoomWindowID

Anyway, is perfect and is exactly what i was looking for

another +1 for trunk

Regards,

Nicolas

[Martin Schönbeck]

+1 here, too

Regards,

Martin

[Nicolas Micoud]

Hi hengsin,

is it ok for you to include that feature in trunk ?

Regards,

Nicolas

[Heng Sin Low]

Yeah, go ahead and create a PR for it.

Actually, there's no need to ask me or Carlos (unless you need input or there's something you are not sure of) before you create any PR and/or Jira tickets. The fact is, usually we can only start a more detailed discussion after you have created a Jira ticket and/or PR (for a better discussion, we need a detailed use case and/or actual code).

To view this discussion on the web visit https://groups.google.com/d/msgid/idempiere/81ed0f74-833e-4567-9356-06fdb7b757fbn%40googlegroups.com.

[Nicolas Micoud]

Hi hengsin

Ok noted ; https://idempiere.atlassian.net/browse/IDEMPIERE-5135