[MODX 3] Media browser uploads file to incorrect path when media source is changed

[halftrainedharry]

Bug report

Summary

Under certain conditions, the media browser uploads files to the wrong path after the media source is changed.

Step to reproduce

• Create a media source. (For example "Images" with basePath/baseUrl = assets/images/.)
• Create a static resource and select a file from the media browser that is located in a subfolder. (For example assets/images/blue.png.)

• Open the media browser again (to select a different file).
• Change the media source. (For example to "Images".)
• Click the "Upload Files" button and select a file.
• Instead of the media source root folder (assets/images/), the file is uploaded to the directory assets/images/assets/images/.

Cause of the error

In the media browser code, when the media source is changed, the active node is set to an empty string: this.cm.activeNode = '';

revolution/manager/assets/modext/widgets/system/modx.tree.directory.js

Lines 270 to 274 in f197da7

	,changeSource: function(sel) {
	this.cm.activeNode = '';
	var s = sel.getValue();
	var rn = this.getRootNode();
	if (rn) { rn.setText(sel.getRawValue()); }

Now when a file is uploaded and the active node is empty, the code uses the value of this.config.openTo for the path (which is set to the path of the file that was selected when the media browser was opened.)

revolution/manager/assets/modext/widgets/system/modx.tree.directory.js

Lines 749 to 751 in f197da7

	,beforeUpload: function() {
	var path = this.config.openTo || this.config.rootId || '/';
	if (this.cm.activeNode) {

Possible solution

In the media browser code, after the media source is changed, set the active node to the root node of the media source.

revolution/manager/assets/modext/widgets/system/modx.tree.directory.js

Lines 273 to 274 in f197da7

	var rn = this.getRootNode();
	if (rn) { rn.setText(sel.getRawValue()); }

if (rn) { rn.setText(sel.getRawValue()); this.cm.activeNode = rn; }

Environment

MODX 3.0.4-pl.

[modxcommunity]

This issue has been mentioned on MODX Community. There might be relevant details there:

https://community.modx.com/t/binary-static-resources-not-downloading/7555/8

[smg6511]

On 3.1.0-dev I'm not seeing this happening. Can you create a video capture of your process of reproducing the issue? Also, a screen grab of the media source setup would help as well.

[halftrainedharry]

On 3.1.0-dev I'm not seeing this happening.

Yes, it looks like this issue is fixed in the current version that's under development.
It seems that with the PR #16394, when the media source is changed, the active node gets set again here in the code:

revolution/manager/assets/modext/widgets/core/tree/modx.tree.js

Line 914 in 4965924

this.cm.activeNode = node