39 replies to this topic

[WanWizard]

Wat ik vermoed is dat door de crash die widget niet wordt aangemaakt, maar zijn child wel. Waarna vervolgens de lookup van die parent resulteert in die crash.

[Pedro_Newbie]

Het toevoegen title="Channel Selection" heeft niet geholpen.

[WanWizard]

Dus die foutmelding over dat "screen_path" heb je ook nog steeds?

[Pedro_Newbie]

Ik denk dat ik het probleem heb gevonden namelijk deze code in de screen RadioInfoBar (start regel 799)

		<widget source="session.CurrentService" render="Picon" position="35,112" size="115,70" alphatest="blend" backgroundColor="secondBG" transparent="1" zPosition="2">
			<convert type="ServiceName">Reference</convert>
		</widget>

Hier loopt ie steeds op stuk, als ik widget weg comment (<!--widget etc.) dan heb ik geen crash
Als ik de code wijzig in

		<widget source="session.CurrentService" render="Picon" position="35,112" size="115,70" alphatest="blend" backgroundColor="secondBG" transparent="1" zPosition="2">
			<convert type="ServiceName">Name</convert>
		</widget>

dan werkt het goed.

Ik weet niet of Reference geldig is of dat het Name moet zijn?

Edit: het vervangen van Reference in Name heeft als effect dat de zender picon niet weergegeven wordt, dat is wel het geval als de radio de eerste keer wordt aangeroepen, de 2e keer crasht ie er op

Edited by Pedro_Newbie, 28 December 2016 - 20:49.

[Rob van der Does]

ServiceName">Reference is de normale manier om picons te vertonen.

In mijn eigen skin komt die ook voor in de radioinfobar.

[Pedro_Newbie]

Ja, maar het is wel een trigger voor de crash, het zal wel veel dieper liggen het probleem dan alleen de skin

[DeBeek]

Als je kijkt in de andere PLi skins komt er alleen bij de PLi-FullnightHD  een picon voor in de radio infobar,

de andere hebben dit allemaal niet, ook de default skin heeft dit niet...

..

[littlesat]

Als die geen picon vind valt er ook niets te renderen.... dus veranderen in name lijkt me ook geen oplossing

[DeBeek]

kijk eens hier...

 

https://github.com/l...eb8a0462a1c1cde

 

helemaal naar beneden scrollen..

Edited by DeBeek, 28 December 2016 - 21:39.

[Pedro_Newbie]

Als die geen picon vind valt er ook niets te renderen.... dus veranderen in name lijkt me ook geen oplossing

Maar er is wel een picon die ook netjes de eerste keer getoond wordt, de 2e aanroep crasht dus

[DeBeek]

Klopt kijk hier maar eens dit had ik in 2014 al geprobeerd..

 

https://forums.openp...ndpost&p=426730

Edited by DeBeek, 28 December 2016 - 21:53.

[Pedro_Newbie]

Inderdaad crasht het ook met TV-Radio-TV pijltje rechts (of links, of omhoog + OK etc.)

[littlesat]

The issue is that the picon is rendered in a 'parentless' instantiateDialog and the eWidget of a picon requires a parent screen....

 

I think it is better to remove the complete RadioInfobar and include all the times from the RadioInfoBar into the ChanelSelectionRadio screen... then you always have the required parent

Edited by littlesat, 29 December 2016 - 00:40.

[littlesat]

Solution (quick) for now...

 

https://github.com/l...914beda4477680b

 

For the real solution we could better remove the complete instantiate window... making all other skins possibly unusable.... So maybe this might be the final fix... I cannnot help that they made an instantiate screen for the RadioInfobar here >10 years ago or so......

 

Skins that did not have a picon yet -or- when no picon was displayed yet because of non-availablitaty for a servers were not effect by this GSOD...

Edited by littlesat, 29 December 2016 - 00:43.

[littlesat]

I hope this finally solves all the paint...

 

https://github.com/l...f42e89a75be7843

 

Layout in line with the rest... and it avoids the GSOD and the RadioInfoBar that was an instantiateDialog Screen is a 'dummy' screen that keeps older skins compatible (no need in python required).

 

Not fully nice as I lifted the orriginal infobar 60 pixels upwards....  So for now some copy and paste, with edit work, inside....

 

And maybe we need to consider to relocate some radio info data.... so the selection list could me make a big larger (later)... but now I have to sleep...

Edited by littlesat, 29 December 2016 - 01:15.

[Pedro_Newbie]

Tenk joe ferry mutz

 

Dit lost het probleem op!

[Rob van der Does]

Merkwaardig: ik heb alvorens te updaten uit alle macht geprobeerd deze crash te reproduceren, maar dat is niet gelukt.

[Rob van der Does]

Overigens na de update van vandaag ook geen chrash

[littlesat]

Opgelost..... oorzaak in de verre achtergrond iets wat ooit 10 jaar geleden of zo verzonnen is...  iets wat onnodig gecompliceerd gemaakt was destijds.... en nu door een picon op het scherm erbij een foutje triggert.. Het liefste sloopte ik dat Instantiate schermpje spul eruit want dat is gewoon onzin... maar daarmee maak ik een boel skins not compatible... dus dat laat ik maar.... daarom staat er wel een dummy scherm in de skin...

 

Er mag uiteraard wel nog e.e.a. gefinetuned worden aan dat scherm... (o.a. RDS op een andere plek ... links naast de kanalenlijst en misschien een normaal font kiezen ervoor... om meer plek te maken voor de radio kanalenlijst...) Misschien laat ik dat finetunen wel aan jullie over...

 

Sorry voor mijn Engels eerder... Ik kreeg van mijn werk een nood telefoontje gisteren avond en kon meteen daarna niet slapen. En dat gesprek was in het Engels en ik zat dus nog in Engels modus...

Edited by littlesat, 29 December 2016 - 09:55.

[Tech]

No problem, apology accepted