[SERVER-25300] Incorrect syntax in for loop in collection.js Created: 27/Jul/16  Updated: 13/Aug/16  Resolved: 08/Aug/16

Status: Closed
Project: Core Server
Component/s: Shell
Affects Version/s: None
Fix Version/s: 3.3.11

Type: Bug Priority: Major - P3
Reporter: Ravind Kumar (Inactive) Assignee: Waley Chen
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
is related to DOCS-8430 your shell javascript has some proble... Closed
Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: Platforms 18 (08/05/16)
Participants:

 Description   

Via DOCS-8430

for loop in collection.js line 683 has incorrect order .

Via jbreams,

The for loop is backwards. It should be

for(i = 0; i < indexSpecs.length; i++)

not

for (i = 0; i++; i < indexSpecs.length)



 Comments   
Comment by Githook User [ 08/Aug/16 ]

Author:

{u'username': u'WaleyChen', u'name': u'Waley Chen', u'email': u'waleycz@gmail.com'}

Message: SERVER-25300 Incorrect syntax in for loop in collection.js
Branch: master
https://github.com/mongodb/mongo/commit/66031378e0dcbb7219ad18bb5494fcddbdf325b8

Comment by Ravind Kumar (Inactive) [ 27/Jul/16 ]

I wouldn't trust myself to fix it, even if it looks simple on its face. Also I have no idea how this syntax error affects the overall functionality, if its had any affect at all. I think its best if the appropriate member of the server team takes a poke at it and goes from there.

Comment by Ramon Fernandez Marina [ 27/Jul/16 ]

ravind.kumar, are you sending a pull request or should we just fix this ourselves?

Generated at Thu Feb 08 04:08:49 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.