Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358942 - Assignment rather than comparison in webdavimpl.js
Summary: Assignment rather than comparison in webdavimpl.js
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.3 RC1   Edit
Assignee: Simon Kaegi CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-26 13:44 EDT by John Arthorne CLA
Modified: 2011-10-07 16:38 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description John Arthorne CLA 2011-09-26 13:44:05 EDT 
webdavImpl.js line 197-200:

	var tail = location.substring(this._rootLocation.length);
	if (tail[tail.length - 1] = "/") {
		tail = tail.substring(0, tail.length - 1);
	}

Surely the if statement should be === rather than =
Comment 1 Simon Kaegi CLA 2011-10-07 16:38:56 EDT 
Woops.Yes indeed, this has been fixed in my local copy for weeks however never pushed the change.