Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SingleLine if/else statements are breaking #15

Closed
segrax opened this issue Jun 25, 2014 · 1 comment
Closed

SingleLine if/else statements are breaking #15

segrax opened this issue Jun 25, 2014 · 1 comment

Comments

@segrax
Copy link

segrax commented Jun 25, 2014

Any single line if statement, is having a semi-colon attached directly after the if evaluation,

examples:

if ( !data.success )
    deferred.reject(); else
    deferred.resolve(data);

becomes

if(!data.success);deferred.reject();else;deferred.resolve(data)}

and

if ( typeof jQuery === 'undefined' )
    throw new Error('.editManager.js: jQuery is required and must be loaded first');

becomes

if(typeof jQuery==='undefined');throw new Error('.editManager.js: jQuery is required and must be loaded first');
matthiasmullie added a commit that referenced this issue Oct 8, 2014
matthiasmullie added a commit that referenced this issue Oct 8, 2014
@matthiasmullie
Copy link
Owner

I've fixed this issue with release 1.3.0.

I've added your tests to the suite as well: 3750e33.
They indeed seem to work fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants