Skip to content


Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP


allow skin.setpath to take a second parameter of the path to search #1590

merged 1 commit into from

2 participants


Simple one. This allows skinners to make buttons on home point to the users' library://video/ nodes.


@cptspiff: This is somewhat needed given the recently added nodes are now using library://video/ paths, yet the skins are still using the old paths. Thoughts?


this is a fix in principle. accept.

@jmarshallnz jmarshallnz was assigned
@jmarshallnz jmarshallnz merged commit 407b515 into from

@ronie want to do the honours and add this to the forum post?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 8, 2012
This page is out of date. Refresh to see the latest.
Showing with 13 additions and 0 deletions.
  1. +13 −0 xbmc/interfaces/Builtins.cpp
13 xbmc/interfaces/Builtins.cpp
@@ -1193,6 +1193,19 @@ int CBuiltins::Execute(const CStdString& execString)
else // execute.Equals("skin.setpath"))
+ if (params.size() > 1)
+ {
+ value = params[1];
+ URIUtils::AddSlashAtEnd(value);
+ bool bIsSource;
+ if (CUtil::GetMatchingSource(value,localShares,bIsSource) < 0) // path is outside shares - add it as a separate one
+ {
+ CMediaSource share;
+ share.strName = g_localizeStrings.Get(13278);
+ share.strPath = value;
+ localShares.push_back(share);
+ }
+ }
if (CGUIDialogFileBrowser::ShowAndGetDirectory(localShares, g_localizeStrings.Get(1031), value))
g_settings.SetSkinString(string, value);
Something went wrong with that request. Please try again.