Skip to content

Commit

Permalink
Merge pull request #17457 from ronie/xbmc-prefix
Browse files Browse the repository at this point in the history
remove support for 'XBMC.' prefixed built-in commands
  • Loading branch information
ronie committed Mar 17, 2020
2 parents 365c4cd + 16518ba commit 3b35ad6
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion addons/service.xbmc.versioncheck/lib/common.py
Expand Up @@ -76,7 +76,7 @@ def get_password_from_user():
return pwd

def message_upgrade_success():
xbmc.executebuiltin("XBMC.Notification(%s, %s, %d, %s)" %(ADDONNAME,
xbmc.executebuiltin("Notification(%s, %s, %d, %s)" %(ADDONNAME,
localise(32013),
15000,
ICON))
Expand Down
2 changes: 0 additions & 2 deletions xbmc/Util.cpp
Expand Up @@ -1040,8 +1040,6 @@ void CUtil::SplitExecFunction(const std::string &execString, std::string &functi

// remove any whitespace, and the standard prefix (if it exists)
StringUtils::Trim(function);
if( StringUtils::StartsWithNoCase(function, "xbmc.") )
function.erase(0, 5);

SplitParams(paramString, parameters);
}
Expand Down
2 changes: 1 addition & 1 deletion xbmc/guilib/GUIButtonControl.dox
Expand Up @@ -35,7 +35,7 @@ action(s) should be performed when pushed.
<textoffsetx></textoffsetx>
<textoffsety></textoffsety>
<pulseonselect></pulseonselect>
<onclick>XBMC.ActivateWindow(MyVideos)</onclick>
<onclick>ActivateWindow(MyVideos)</onclick>
<onfocus>-</onfocus>
<onunfocus>-</onunfocus>
<onup>2</onup>
Expand Down
1 change: 0 additions & 1 deletion xbmc/interfaces/builtins/Builtins.cpp
Expand Up @@ -142,7 +142,6 @@ void CBuiltins::GetHelp(std::string &help)

int CBuiltins::Execute(const std::string& execString)
{
// Deprecated. Get the text after the "XBMC."
std::string execute;
std::vector<std::string> params;
CUtil::SplitExecFunction(execString, execute, params);
Expand Down

0 comments on commit 3b35ad6

Please sign in to comment.