Skip to content

I
InstiApp
Commit 1aef2ad4 authored by Varun Patil's avatar Varun Patil
Browse files
Options

Dont' set listeners to invisible objects

parent d390c39b
Show whitespace changes
Inline Side-by-side
Showing with 10 additions and 10 deletions
app/src/main/java/in/ac/iitb/gymkhana/iitbapp/fragment/EventFragment.java
View file @ 1aef2ad4
...@@ -147,10 +147,9 @@ public class EventFragment extends BaseFragment implements View.OnClickListener ...@@ -147,10 +147,9 @@ public class EventFragment extends BaseFragment implements View.OnClickListener
startActivity(Intent.createChooser(i, "Share URL")); startActivity(Intent.createChooser(i, "Share URL"));
} }
}); });
if (event.getEventWebsiteURL() != null) if (event.getEventWebsiteURL() != null && !event.getEventWebsiteURL().isEmpty())
{ {
webEventButton.setVisibility(View.VISIBLE); webEventButton.setVisibility(View.VISIBLE);
}
webEventButton.setOnClickListener(new View.OnClickListener() { webEventButton.setOnClickListener(new View.OnClickListener() {
String eventwebURL = event.getEventWebsiteURL(); String eventwebURL = event.getEventWebsiteURL();
@Override @Override
...@@ -160,6 +159,7 @@ public class EventFragment extends BaseFragment implements View.OnClickListener ...@@ -160,6 +159,7 @@ public class EventFragment extends BaseFragment implements View.OnClickListener
} }
}); });
} }
}
@Override @Override
public void onClick(View view) { public void onClick(View view) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment