Skip to content

Commit

Permalink
revert DataTables host page name to gridviewer.html
Browse files Browse the repository at this point in the history
RStudio stores this HTML page name in the contentUrl, so changing it
could break existing projects.
  • Loading branch information
jmcphers committed Dec 1, 2017
1 parent 976735a commit fcad56c
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/cpp/session/modules/data/DataViewer.cpp
Expand Up @@ -344,7 +344,7 @@ json::Value makeDataItem(SEXP dataSEXP, const std::string& caption,
dataItem["cacheKey"] = cacheKey;
dataItem["object"] = objName;
dataItem["environment"] = envName;
dataItem["contentUrl"] = kGridResource "/dtviewer.html?env=" +
dataItem["contentUrl"] = kGridResource "/gridviewer.html?env=" +
http::util::urlEncode(envName, true) + "&obj=" +
http::util::urlEncode(objName, true) + "&cache_key=" +
http::util::urlEncode(cacheKey, true);
Expand Down
Expand Up @@ -24,7 +24,7 @@ public class GridViewerFrame extends RStudioFrame
{
public GridViewerFrame()
{
super("grid_resource/dtviewer.html?data_source=data");
super("grid_resource/gridviewer.html?data_source=data");
}

public void onAttach()
Expand Down

0 comments on commit fcad56c

Please sign in to comment.