Reply to thread

Message

<blockquote data-quote="CyberSimian" data-source="post: 1254279" data-attributes="member: 141969">First, apologies to [USER=68833]@ajs[/USER]. The sample code that I suggested be placed in &quot;BasicHome.xml&quot; does work correctly.&nbsp; Looking at it this morning, I see that what I tested was the visibility condition &quot;Player.Recording&quot;.&nbsp; As there is some doubt about that visibility condition working, let us put that aside for now and consider only &quot;#TV.Record.Active&quot;.This is what testing this morning has shown:(1) If I start MP with a recording active, &quot;#TV.Record.Active&quot; displays as the value &quot;true&quot; on the &quot;Basic Home&quot; panel. <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" />(2) If I start MP with no recording active, &quot;#TV.Record.Active&quot; displays as the null string. <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite3" alt=":(" title="Frown&nbsp; &nbsp; :(" loading="lazy" data-shortname=":(" /> I believe that this is the incorrect value -- it should display as the string &quot;false&quot;.I would suspect that (2) arises because &quot;#TV.Record.Active&quot; is being initialised to the null string (zero length string). The null string is an appropriate initial value for many built-in skin variables, but not in this case.&nbsp; The initial value in this case should be the string &quot;false&quot;.This area is so complicated that we need to fix one bug at a time.&nbsp; Testing with multiple bugs present just leads to confusion. Yes, I think that there are other bugs present, but we will get to those when this initial-value bug has been fixed.[USER=68833]@ajs[/USER] If you agree with the change to the initial value, please make that change and create another Team-City build for me to test. Thank you.-- from CyberSimian in the UK</blockquote>

[QUOTE="CyberSimian, post: 1254279, member: 141969"] First, apologies to [USER=68833]@ajs[/USER]. The sample code that I suggested be placed in "BasicHome.xml" [b]does work correctly[/b]. Looking at it this morning, I see that what I tested was the visibility condition "Player.Recording". As there is some doubt about that visibility condition working, let us put that aside for now and consider only "#TV.Record.Active". This is what testing this morning has shown: (1) If I start MP with a recording active, "#TV.Record.Active" displays as the value "true" on the "Basic Home" panel. :) (2) If I start MP with no recording active, "#TV.Record.Active" displays as the null string. :( I believe that this is the incorrect value -- it should display as the string "false". I would suspect that (2) arises because "#TV.Record.Active" is being initialised to the null string (zero length string). The null string is an appropriate initial value for many built-in skin variables, but not in this case. The initial value in this case should be the string "false". This area is so complicated that we need to fix one bug at a time. Testing with multiple bugs present just leads to confusion. Yes, I [i]think[/i] that there are other bugs present, but we will get to those when this initial-value bug has been fixed. [USER=68833]@ajs[/USER] If you agree with the change to the initial value, please make that change and create another Team-City build for me to test. Thank you. -- from CyberSimian in the UK [/QUOTE]

Verification