Bug - Regression: "From" and "To" can not be expanded if an article is loaded after opening TIcketZoom

[olifre]

Environment

• Server OS: Linux
• Browser: Tested with Firefox and Chromium
• Znuny version: since 7.0.8

Expected behavior

Clicking on "From" and "To" in an article should expand the full mail address, i.e.:

John Doe

should be expanded to:

John Doe <[email protected]>

(and unexpanded again when clicking again).

Actual behavior

This works if a ticket is visited directly, i.e.:
https://someserver.example.com/znuny/index.pl?Action=AgentTicketZoom;TicketID=1234
but fails either if:

• An article is clicked upon first with the mouse (can be the exact same article).
• A direct article URL is used, i.e.:
  https://someserver.example.com/znuny/index.pl?Action=AgentTicketZoom;TicketID=1234#2345

How to reproduce

Steps to reproduce the behavior:

1. Visit any ticket which has some name and mail address in From / To.
2. Click on "From" or "To".
3. Observe it is expanded.
4. Visit the ticket via an article URL or click on an article to load it.
5. Observe it is not expandable anymore.

Additional information

This is a regression caused by:
4f3baef
which adapted the template to use div instead of p, but only adapted the javascript code in:

Znuny/var/httpd/htdocs/js/Core.Agent.TicketZoom.js

Lines 787 to 790 in 86309bb

	// add switchable toggle
	$('label.Switchable').off('click.Switch').on('click.Switch', function() {
	$(this).next('div').find('.Switch').toggleClass('Hidden');
	});

(i.e. Init of Core.Agent.TicketZoom), but not in:

Znuny/var/httpd/htdocs/js/Core.Agent.TicketZoom.js

Lines 213 to 216 in 86309bb

	// add switchable toggle for new article
	$('label.Switchable').off('click.Switch').on('click.Switch', function() {
	$(this).next('p.Value').find('.Switch').toggleClass('Hidden');
	});

(i.e. LoadArticle of Core.Agent.TicketZoom).

Adapting the second location should fix the problem 😉 .

[dignin]

Internal Issue# 991