res_pjsip_pidf_eyebeam_body_supplement: Correct status presentation
authorSean Bright <sean.bright@gmail.com>
Wed, 26 Jul 2017 13:48:29 +0000 (09:48 -0400)
committerGeorge Joseph <gjoseph@digium.com>
Tue, 1 Aug 2017 21:42:38 +0000 (15:42 -0600)
commit2be8d91c0f5802a1f9c87b35f8f419e0db1b22d8
treee412f590ea4cede5ef0cf8f5f1f6ac3ee78e461b
parent2a4283f3e7f0c8459be1f6e877d3ef45a8ceacc2
res_pjsip_pidf_eyebeam_body_supplement: Correct status presentation

This change fixes PIDF content generation when the underlying device
state is considered in use. Previously it was incorrectly marked
as closed meaning they were offline/unavailable. The code now
correctly marks them as open.

Additionally:

  * Generate an XML element for our activity instead of a using a text
    node.

  * Consider every extension state other than "unavailable" to be 'open'
    status.

  * Update the XML namespaces and structure to reflect those
    documented in RFC 4480

  * Use 'on-the-phone' (defined in RFC 4880) instead of 'busy' as the
    "in use" activity. This change results in eyeBeam using the
    appropriate icon for the watched user.

This was tested on eyeBeam 1.5.20.2 build 59030 on Windows.

ASTERISK-26659 #close
Reported by: Abraham Liebsch
patches:
  ASTERISK-26659.diff submitted by snuffy (license 5024)

Change-Id: I6e5ad450f91106029fb30517b8c0ea0c2058c810
res/res_pjsip/presence_xml.c
res/res_pjsip_pidf_body_generator.c
res/res_pjsip_pidf_eyebeam_body_supplement.c