Is there Any JavaScript method to stop a function from working

Question

I have a function which has a if statement and an onclick function. I wanted that when the if statement is true than the whole function should not work. I have tried to do return false and it didn't work for me

showLogo: function(row) {
  var a = 0;
  window.onclick = (e => {
    let element = e.target.className || e.srcElement.className
    let element1 = e.target || e.srcElement
    if (element1.localName == "li" || element == "pagination-previous" || element == "pagination-next" || element === "pagination-link" || element === "icon" || element === "fa fa-angle-right fa-lg" || element === "b-table" || element === "level" || element === "pagination" || element === "fa fa-angle-left fa-lg") {
      this.logo = false
    }
  });
  if (window.innerWidth > 1024) {
    this.logo = true;
    this.color = false;
    this.istabModalActive = false;
    this.name = row.fileName;
    this.dimension = row.width + ' x ' + row.height
    this.width = 0;
    this.height = 0;
    this.src = row.path;
    this.icon = row.iconsrc;
    this.dlsrc = row.path;
    this.contentType = row.contentType;
    this.typeshow = row.contentType.split("/")[1].toUpperCase();
    this.type = row.contentType.split("/")[1];
    this.icons = 'dist/' + this.type + '.svg';
    this.assetsid = row.index;
    this.size = row.size;
    this.activerow = row;
    this.fileid = row.id;
    this.assetsid = row.id;
    this.activerow = row;
  } else {
    this.logo = true;
    this.color = false;
    this.istabModalActive = true;
    this.filesarr = row;
    this.name = row.fileName;
    this.dimension = row.width + ' x ' + row.height
    this.size = row.size;
    this.src = row.path;
    this.icon = row.iconsrc;
    this.dlsrc = row.path;
    this.typeshow = row.contentType.split("/")[1].toUpperCase();
    this.type = row.contentType.split("/")[1];
    this.icons = row.iconsrc;
    this.fileid = row.id;
    this.assetsid = row.id;
    this.activerow = row;
  }

}

Possible duplicate of [Early exit from function?](https://stackoverflow.com/questions/3330193/early-exit-from-function) — Muhammet Can TONBUL, Dec 29 '17 at 06:32
i have edited my post please check the full function that i am wokring with and i have use the return but it didn't work — Haroon Aslam, Dec 29 '17 at 06:37
@HaroonAslam But the function won't execute any further soon as you `return` from it. Isn't that what you want? — vibhor1997a, Dec 29 '17 at 06:41
i have used the return but is is still working as it was before — Haroon Aslam, Dec 29 '17 at 06:42
It will execute till the return statement as you are already invoking it so it would be called. — vibhor1997a, Dec 29 '17 at 06:43
@HaroonAslam did you place that `return` statement on the top of your function body? — techie_28, Dec 29 '17 at 06:47
@vibhor1997a didn't understand that how can i move the `window.onlick`in else — Haroon Aslam, Dec 29 '17 at 06:53
@HaroonAslam Aren't you referring to `if(window.innerWidth> 1024)` if statement in your question? — vibhor1997a, Dec 29 '17 at 07:01

score 1 · Answer 1 · answered Dec 29 '17 at 06:43

showLogo:function(row){
    var a = 0;
    var flag = 0;
    window.onclick = (e => {
        let element = e.target.className || e.srcElement.className
        let element1 = e.target || e.srcElement
        if (element1.localName == "li" || element == "pagination-previous" || element == "pagination-next" || element === "pagination-link" ||  element === "icon" ||  element === "fa fa-angle-right fa-lg" ||  element === "b-table"||  element === "level"||  element === "pagination" ||  element === "fa fa-angle-left fa-lg") {
            this.logo = false;
            flag = 1;
        }


        }); 
                if(window.innerWidth> 1024 && flag == 0)
                {
                this.logo = true;
                this.color = false; 
                this.istabModalActive = false;
                this.name = row.fileName;
                this.dimension = row.width +' x '+row.height
                this.width = 0;
                this.height = 0;
                this.src = row.path;
                this.icon = row.iconsrc;
                this.dlsrc = row.path;
                this.contentType = row.contentType;
                this.typeshow = row.contentType.split("/")[1].toUpperCase() ;
                this.type = row.contentType.split("/")[1];
                this.icons = 'dist/'+this.type+'.svg';
                this.assetsid=row.index;
                this.size = row.size;
                this.activerow = row;
                this.fileid=row.id;
                this.assetsid=row.id;
                this.activerow = row;
                }
                else if(flag == 0) {
                this.logo = true;
                this.color = false; 
                this.istabModalActive = true;
                this.filesarr=row;
                this.name = row.fileName;
                this.dimension = row.width +' x '+row.height
                this.size = row.size;
                this.src = row.path;
                this.icon = row.iconsrc;
                this.dlsrc = row.path;
                this.typeshow = row.contentType.split("/")[1].toUpperCase() ;
                this.type = row.contentType.split("/")[1];
                this.icons = row.iconsrc;
                this.fileid=row.id;
                this.assetsid=row.id;
                this.activerow = row;   
                }

},

yes but didn't work cause the flag in the onclick is not getting out of the `window.onclick` function i have used alert to find out — Haroon Aslam, Dec 29 '17 at 07:00
i have done what you said something was achieved but on the first attempt it is wokring that it should not and after that it is working like i wanted it to be — Haroon Aslam, Dec 29 '17 at 07:07
and not the window is not opening after the second attempt with onlick funciton it is not working properly — Haroon Aslam, Dec 29 '17 at 07:09
assign `flag = 0` below this line `et element1 = e.target || e.srcElement` — Munasir Pv, Dec 29 '17 at 07:10

score 0 · Answer 2 · answered Dec 29 '17 at 07:33

setWindowOnCLick : function() {
  window.onclick = (e => {
        let element = e.target.className || e.srcElement.className
        let element1 = e.target || e.srcElement
        if (element1.localName == "li" || element == "pagination-previous" || element == "pagination-next" || element === "pagination-link" || element === "icon" || element === "fa fa-angle-right fa-lg" || element === "b-table" || element === "level" || element === "pagination" || element === "fa fa-angle-left fa-lg") {
          this.logo = false
        }
      });
}  
showLogo: function(row) {
      var a = 0;

      if (window.innerWidth > 1024) {
        this.logo = true;
        this.color = false;
        this.istabModalActive = false;
        this.name = row.fileName;
        this.dimension = row.width + ' x ' + row.height
        this.width = 0;
        this.height = 0;
        this.src = row.path;
        this.icon = row.iconsrc;
        this.dlsrc = row.path;
        this.contentType = row.contentType;
        this.typeshow = row.contentType.split("/")[1].toUpperCase();
        this.type = row.contentType.split("/")[1];
        this.icons = 'dist/' + this.type + '.svg';
        this.assetsid = row.index;
        this.size = row.size;
        this.activerow = row;
        this.fileid = row.id;
        this.assetsid = row.id;
        this.activerow = row;
      } else {
        this.logo = true;
        this.setWindowOnCLick.call(this)
        //thismay solve your problem
        this.color = false;
        this.istabModalActive = true;
        this.filesarr = row;
        this.name = row.fileName;
        this.dimension = row.width + ' x ' + row.height
        this.size = row.size;
        this.src = row.path;
        this.icon = row.iconsrc;
        this.dlsrc = row.path;
        this.typeshow = row.contentType.split("/")[1].toUpperCase();
        this.type = row.contentType.split("/")[1];
        this.icons = row.iconsrc;
        this.fileid = row.id;
        this.assetsid = row.id;
        this.activerow = row;
      }

    }

thanks but not working I tweaked with this a little bit but not getting anywhere — Haroon Aslam, Dec 29 '17 at 12:13

Is there Any JavaScript method to stop a function from working

2 Answers2