💫improve performance.

Restrict unnecessary api calls

Author: TheAlphamerc

lib/bloc/bloc/repo_event.dart

@@ -58,14 +58,14 @@ class LoadForksEvent extends RepoEvent {
   final bool isLoadNextForks;
   final String name;
   final String owner;
-  LoadForksEvent({this.isLoadNextForks=false,this.name,this.owner}) : assert(name != null), assert(owner!= null);
+  final int count;
+  LoadForksEvent({this.name,this.owner,this.count,this.isLoadNextForks=false}) : assert(name != null), assert(owner!= null);
   @override
   Stream<RepoState> getRepoForks(
       {RepoState currentState, RepoBloc bloc}) async* {
     try {
       yield LoadingForksState();
-
-      final list = await _repoRepository.fetchRepoForks(name: name, owner: owner);
+      final list = count == 0 ? null : await _repoRepository.fetchRepoForks(name: name, owner: owner);
       yield LoadedForksState(list);
     } catch (_, stackTrace) {
       developer.log('$_',name: 'LoadForksEvent', error: _, stackTrace: stackTrace);

lib/bloc/gist/gist_event.dart

@@ -31,11 +31,11 @@ class LoadGistDetailEvent extends GistEvent {
 }
 
 class OnGistLoad extends GistEvent {
-  OnGistLoad(this.login, {this.isLoadNextGist = false});
+  OnGistLoad(this.login, {this.count,this.isLoadNextGist = false});
 
   final bool isLoadNextGist;
   final String login;
-
+  final int count;
   @override
   Stream<GistState> getGist(
       {GistState currentState, GistBloc bloc}) async* {
@@ -45,7 +45,7 @@ class OnGistLoad extends GistEvent {
     try {
       // yield LoadingUserState();
 
-      final gistModel = await _gistRepository.fetchGistList(login: login);
+      final gistModel = count == 0 ? null : await _gistRepository.fetchGistList(login: login);
 
       yield LoadedGitState(gist: gistModel);
     } catch (_, stackTrace) {

lib/bloc/gist/gist_state.dart

@@ -38,6 +38,8 @@ class LoadedGitState extends GistState {
     print("New cursor is ${gistModel.pageInfo.endCursor}");
     return LoadedGitState(gist: currenctGistModel,);
   }
+
+  bool get isNotNullEmpty => this.gist != null && this.gist.totalCount > 0;
 }
 
 class LoadingNextGistState extends LoadedGitState {

lib/bloc/issues/issues_event.dart

@@ -22,9 +22,10 @@ abstract class IssuesEvent extends Equatable {
 
 class LoadIssuesEvent extends IssuesEvent {
   final String login;
+  final int count;
   final bool isLoadNextIssues;
 
-  LoadIssuesEvent(this.login, {this.isLoadNextIssues = false});
+  LoadIssuesEvent(this.login, {this.count,this.isLoadNextIssues = false});
   @override
   Stream<IssuesState> loadIssues(
       {IssuesState currentState, IssuesBloc bloc}) async* {
@@ -33,7 +34,7 @@ class LoadIssuesEvent extends IssuesEvent {
         return;
       }
       yield LoadingUserIssuesState();
-      final issues = await _issuesRepository.getIssues(login: login);
+      final issues =count ==0 ? null : await _issuesRepository.getIssues(login: login);
       yield LoadedIssuesState(issues);
     } catch (_, stackTrace) {
       developer.log('$_',
@@ -70,9 +71,10 @@ class LoadIssuesEvent extends IssuesEvent {
 class LoadRepoIssuesEvent extends IssuesEvent {
   final String owner;
   final String name;
+  final int count;
   final bool isLoadNextRepoIssues;
 
-  LoadRepoIssuesEvent({this.owner,this.name, this.isLoadNextRepoIssues = false});
+  LoadRepoIssuesEvent({this.owner,this.name,this.count,this.isLoadNextRepoIssues = false});
   @override
   Stream<IssuesState> loadRepoIssues(
       {IssuesState currentState, IssuesBloc bloc}) async* {
@@ -81,7 +83,7 @@ class LoadRepoIssuesEvent extends IssuesEvent {
         return;
       }
       yield LoadingRepoIssuesState();
-      final issues = await _issuesRepository.getRepoIssues(name:name,owner:owner);
+      final issues = count == 0 ? null : await _issuesRepository.getRepoIssues(name:name,owner:owner);
       yield LoadedRepoIssuesState(issues);
     } catch (_, stackTrace) {
       developer.log('$_',

lib/bloc/issues/issues_state.dart

@@ -59,6 +59,9 @@ class LoadedRepoIssuesState extends IssuesState {
     currentIssueModel.pageInfo = issuesModel.pageInfo;
     return LoadedRepoIssuesState(currentIssueModel);
   }
+
+  bool get isNotNullEmpty => this.issues != null && this.issues.list != null &&
+                this.issues.list.isNotEmpty;
 }
 
 class ErrorRepoIssuesState extends IssuesState {

lib/bloc/notification/notification_event.dart

@@ -31,8 +31,7 @@ class OnLoad extends NotificationEvent {
       }
       yield LoadingNotificationState();
       final list = await _userRepository.getNotificationsList(pageNo: 1);
-      yield LoadedNotificationState(
-          list: list, pageNo: 2, hasNextPage: list.length == 10);
+      yield LoadedNotificationState(list: null, pageNo: 2, hasNextPage: list.length == 10);
     } catch (_, stackTrace) {
       developer.log('$_',
           name: 'Load Repository Event', error: _, stackTrace: stackTrace);
@@ -48,8 +47,7 @@ class OnLoad extends NotificationEvent {
         print("No notification left");
         return;
       }
-      yield LoadingNextNotificationState(
-          state.list, state.pageNo, state.hasNextPage);
+      yield LoadingNextNotificationState(state.list, state.pageNo, state.hasNextPage);
       final list = await _userRepository.getNotificationsList(pageNo: state.pageNo);
       yield LoadedNotificationState.next(
         currentList: state.list,

lib/bloc/people/people_event.dart

@@ -84,20 +84,20 @@ class LoadUserEvent extends PeopleEvent {
 }
 
 class LoadFollowEvent extends PeopleEvent {
-  LoadFollowEvent(this.login, this.type, {this.isLoadNextFollow = false});
+  LoadFollowEvent(this.login, this.type, {this.count, this.isLoadNextFollow = false});
 
   final bool isLoadNextFollow;
   final String login;
   final PeopleType type;
+  final int count;
 
   @override
   Stream<PeopleState> fetchFollowersList(
       {PeopleState currentState, PeopleBloc bloc}) async* {
     try {
       if (!isLoadNextFollow) {
         yield LoadingFollowState();
-        final followers =
-            await _peopleRepository.fetchFollowersList(login, type: type);
+        final followers = count == 0 ? null :  await _peopleRepository.fetchFollowersList(login, type: type);
         yield LoadedFollowState(followers);
       } else {
         final state = currentState as LoadedFollowState;
@@ -108,8 +108,7 @@ class LoadFollowEvent extends PeopleEvent {
         yield LoadingNextFollowState(state.followModel);
         final followModel = await _peopleRepository.fetchFollowersList(login,
             type: type, endCursor: state.followModel.pageInfo.endCursor);
-        yield LoadedFollowState.next(
-            model: followModel, currentFollowModel: state.followModel);
+        yield LoadedFollowState.next(model: followModel, currentFollowModel: state.followModel);
       }
     } on OperationException catch (_, stackTrace) {
       developer.log('$_',
@@ -174,14 +173,15 @@ class LoadWatchersEvent extends PeopleEvent {
   final bool isLoadNextWatchers;
   final String name;
   final String owner;
-  LoadWatchersEvent({this.isLoadNextWatchers=false,this.name,this.owner}) : assert(name != null), assert(owner!= null);
+  final int count;
+  LoadWatchersEvent({this.isLoadNextWatchers=false,this.count,this.name,this.owner}) : assert(name != null), assert(owner!= null);
   @override
   Stream<PeopleState> getRepoWatchers(
       {PeopleState currentState, PeopleBloc bloc}) async* {
     try {
       yield LoadingWatcherState();
 
-      final list = await _peopleRepository.getRepoWatchers(name: name, owner: owner);
+      final list = count == 0 ? null : await _peopleRepository.getRepoWatchers(name: name, owner: owner);
       yield LoadedWatcherState(list);
     } catch (_, stackTrace) {
       developer.log('$_',
@@ -213,14 +213,14 @@ class LoadStargezersEvent extends PeopleEvent {
   final bool isLoadNextStartgers;
   final String name;
   final String owner;
-  LoadStargezersEvent({this.isLoadNextStartgers=false,this.name,this.owner}) : assert(name != null), assert(owner!= null);
+  final int count;
+  LoadStargezersEvent({this.name,this.owner, this.count,this.isLoadNextStartgers=false}) : assert(name != null), assert(owner!= null);
   @override
   Stream<PeopleState> getRepoStargezres(
       {PeopleState currentState, PeopleBloc bloc}) async* {
     try {
       yield LoadingStargezersState();
-
-      final list = await _peopleRepository.fetchRepoStargazers(name: name, owner: owner);
+      final list = count == 0 ? null : await _peopleRepository.fetchRepoStargazers(name: name, owner: owner);
       yield LoadedStargezersState(list);
     } catch (_, stackTrace) {
       developer.log('$_',name: 'LoadStargezersEvent', error: _, stackTrace: stackTrace);

lib/bloc/people/people_state.dart

@@ -155,6 +155,9 @@ class LoadedStargezersState extends PeopleState {
    currentStargazers.pageInfo = stargezers.pageInfo;
    return LoadedStargezersState(currentStargazers);
   }
+
+  bool get isNotNullEmpty => this.stargezers != null && this.stargezers.userList != null &&
+                this.stargezers.userList.isNotEmpty;
 }
 class ErrorStargezersState extends LoadedUserState {
   final String errorMessage;

lib/bloc/pullrequest/pullrequest_event.dart

@@ -12,18 +12,19 @@ abstract class PullrequestEvent extends Equatable{
 class LoadRepoPullRequests extends PullrequestEvent {
   final String owner;
   final String name;
+  final int count;
   final bool isNextRepoPull;
 
 
-  LoadRepoPullRequests({this.name, this.owner,this.isNextRepoPull = false}) : assert(owner != null),assert(name!= null);
+  LoadRepoPullRequests({this.name, this.owner,this.count,this.isNextRepoPull = false}) : assert(owner != null),assert(name!= null);
 
   @override
   Stream<PullrequestState> getrepoPullrequest(
       {PullrequestState currentState, PullrequestBloc bloc}) async* {
     try {
       yield LoadingRepoPullRequest();
 
-      final repoPullRequest = await _pullrequestRepository.fetchRepoPullRequest(name:name, owner:owner);
+      final repoPullRequest = count == 0 ? null : await _pullrequestRepository.fetchRepoPullRequest(name:name, owner:owner);
       yield LoadedRepoPullRequest(repoPullRequest: repoPullRequest);
     } catch (_, stackTrace) {
       developer.log('$_', name: 'LoadPullrequestEvent', error: _, stackTrace: stackTrace);
@@ -57,11 +58,11 @@ class LoadRepoPullRequests extends PullrequestEvent {
 }
 
 class OnPullRequestLoad extends PullrequestEvent {
-  OnPullRequestLoad(this.login, {this.isLoadNextIssues = false});
+  OnPullRequestLoad(this.login, {this.count,this.isLoadNextIssues = false});
 
   final bool isLoadNextIssues;
   final String login;
-
+  final int count;
   @override
   Stream<PullrequestState> getPullRequest(
       {PullrequestState currentState, PullrequestBloc bloc}) async* {
@@ -70,8 +71,7 @@ class OnPullRequestLoad extends PullrequestEvent {
     }
     try {
       yield LoadingPullRequestState();
-      final pullRequestsList =
-          await _pullrequestRepository.fetchUserPullRequest(login:login);
+      final pullRequestsList = count == 0 ? null : await _pullrequestRepository.fetchUserPullRequest(login:login);
       yield LoadedPullRequestState(pullRequestsList: pullRequestsList);
     } catch (_, stackTrace) {
       developer.log('$_',

lib/ui/page/home/widgets/home_page_screen.dart

@@ -126,7 +126,7 @@ class HomePageScreenBody extends StatelessWidget {
         children: <Widget>[
           _getUtilRos(context, GIcons.issue_opened_24, "Issues",
               color: GColors.green, onPressed: () {
-            Navigator.push(context, IssuesPage.route(model.login));
+            Navigator.push(context, IssuesPage.route(model.login,count:model.issues.totalCount));
           }),
           Divider(height: 0, indent: 50),
           _getUtilRos(context, GIcons.git_pull_request_16, "Pull Request",