Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 496736

Summary:

Usage of .prototype.contructor instead of .prototype.constructor.

Product:

[ECD] Orion

Reporter:

Olivier Thomann <Olivier_Thomann>

Component:

JS Tools

Assignee:

Olivier Thomann <Olivier_Thomann>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Version:

12.0

Target Milestone:

13.0

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
Proposed patch	none

Description Olivier Thomann

2016-06-24 10:16:52 EDT

I found this typo in:
bundles/org.eclipse.orion.client.javascript/web/javascript/hover.js
bundles/org.eclipse.orion.client.javascript/web/javascript/occurrences.js
bundles/org.eclipse.orion.client.javascript/web/javascript/quickFixes.js
bundles/org.eclipse.orion.client.javascript/web/webtools/cssHover.js
bundles/org.eclipse.orion.client.javascript/web/webtools/cssQuickFixes.js
bundles/org.eclipse.orion.client.javascript/web/webtools/htmlHover.js
bundles/org.eclipse.orion.client.javascript/web/webtools/htmlOccurrences.js

Comment 1 Olivier Thomann

2016-06-24 10:17:12 EDT

Created attachment 262677 [details]
Proposed patch

Comment 2 Mark Macdonald

2016-06-24 10:21:32 EDT

Is there any point to setting the constructor property at all? It doesn't seem to be used anywhere in the code base. Seems like we could just remove the misspelled lines completely.

Comment 3 Olivier Thomann

2016-06-24 11:57:14 EDT

ok, I'll do it.

Comment 4 Olivier Thomann

2016-06-24 12:06:24 EDT

Delivered. Removed the lines.