Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check if charsetSetting is null before trying to get data from it. #8086

Merged
merged 1 commit into from Oct 4, 2015

Conversation

razzeee
Copy link
Member

@razzeee razzeee commented Sep 19, 2015

If it is use the default value.

Fixes http://forum.kodi.tv/showthread.php?tid=229292&pid=2109962#pid2109962
But will reset your profiles.xml if it is not valid xml.

@MartijnKaijser
Copy link
Member

i think we always reset to default if something went wrong. @Montellese would probably know

@mkortstiege mkortstiege added the Type: Fix non-breaking change which fixes an issue label Oct 4, 2015
@razzeee razzeee added this to the Jarvis 16.0-alpha4 milestone Oct 4, 2015
razzeee added a commit that referenced this pull request Oct 4, 2015
Check if charsetSetting is null before trying to get data from it.
@razzeee razzeee merged commit a67e452 into xbmc:master Oct 4, 2015
@razzeee razzeee deleted the fix-invalid-profiles-xml branch October 21, 2015 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Fix non-breaking change which fixes an issue v16 Jarvis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants