Code cleaning deletes JavaScript references

[Mark Ewer]

I love the code cleaning feature of JustCode, but I have found a problem with it.  In Visual Studio you use a comment that starts with three slashes.  But, if you turn on the code cleaning feature to introduce header text, it deletes all of the references!  And, of course, this breaks the JavaScript IntelliSense.

So, I had this:

     /// <reference path="../../Scripts/_references.js" />
     /// <reference path="../app.js" />'use strict';
     var menuCtrl = clippyApp.controller('menuCtrl');

but JustCode turned it into this when I chose the code cleaning option

     // HEADER TEXT
     var menuCtrl = clippyApp.controller('menuCtrl');

All of the <reference> lines where deleted from the file.

[Svetlozar]

Hi,

Thank you for your feedback and sorry for the inconvenience!

If you execute Introduce Header and there are existing comments in the file JustCode will remove them, because there is no way to know if these are old header text you want to remove or meaningful comments. 

However, we can assume that JavaScript references are meaningful comments that should stay intact. We have already fixed that, so please, stay tuned for our next release.

Most probably we will add an option whether we should remove or leave existing comments on Introduce Header, what do you think? 

Regards,
Svetlozar
Telerik

 

 

[Mark Ewer]

I expected it to remove existing comments ... but the reference lines are not comments!  I don't want to run into the situation where it has multiple header blocks at the top of the file.  In my opinion, removing the comments from the top of the file makes sense, but the JavaScript references are different. 

[Svetlozar]

Hello,

This is what we did, Introduce Header now removes all comments except for JavaScript references and introduces the header. It will be included in our next release, so please, stay tuned!

If you have any other problems or suggestions, please don't hesitate to write back.

Regards,
Svetlozar
Telerik