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

Minor cleanup to match naming convention in video database #8213

Merged
merged 1 commit into from
Oct 18, 2015
Merged

Minor cleanup to match naming convention in video database #8213

merged 1 commit into from
Oct 18, 2015

Conversation

xconverge
Copy link
Contributor

To complement [xbmc][library] Fix export library #8152

@mkortstiege
Copy link
Member

@Paxxi your button ;)

@mkortstiege mkortstiege added the Type: Cleanup non-breaking change which removes non-working or unmaintained functionality label Oct 13, 2015
@xconverge
Copy link
Contributor Author

This export still has some issues. The separate files export doesn't seem to work correctly to the folder you choose, I was looking at it a bit more

@Paxxi
Copy link
Member

Paxxi commented Oct 13, 2015

This will break as singlefiles means one file per thing and singlefile means everything in one file so the checks need to be reversed. E.g if(singlefiles) becomes if(!singlefile) and so on, if you can update it we can merge it

@xconverge
Copy link
Contributor Author

You are completely right, I will make the changes

To complement [xbmc][library] Fix export library #8152
@xconverge
Copy link
Contributor Author

Updated @Paxxi feel free to let me know if anything is still broken.

@mkortstiege
Copy link
Member

Looks good to me. @Paxxi please merge when you're happy with it.

@Paxxi
Copy link
Member

Paxxi commented Oct 18, 2015

Haven't had time to test it but changes look good

Paxxi added a commit that referenced this pull request Oct 18, 2015
Minor cleanup to match naming convention in video database
@Paxxi Paxxi merged commit e5771b5 into xbmc:master Oct 18, 2015
@hudokkow hudokkow added this to the Jarvis 16.0-alpha4 milestone Oct 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Cleanup non-breaking change which removes non-working or unmaintained functionality v16 Jarvis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants