Skip to content

Commit f801024

Browse files
committed
Use helpers from parent class for setting view fields
1 parent af531f9 commit f801024

File tree

9 files changed

+44
-43
lines changed

9 files changed

+44
-43
lines changed

app/src/main/java/com/github/mobile/ui/commit/CommitCompareListFragment.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import com.actionbarsherlock.view.Menu;
3535
import com.actionbarsherlock.view.MenuInflater;
3636
import com.actionbarsherlock.view.MenuItem;
37+
import com.github.kevinsawicki.wishlist.ViewFinder;
3738
import com.github.kevinsawicki.wishlist.ViewUtils;
3839
import com.github.mobile.R.id;
3940
import com.github.mobile.R.layout;
@@ -57,8 +58,6 @@
5758
import org.eclipse.egit.github.core.RepositoryCommit;
5859
import org.eclipse.egit.github.core.RepositoryCommitCompare;
5960

60-
import roboguice.inject.InjectView;
61-
6261
/**
6362
* Fragment to display a list of commits being compared
6463
*/
@@ -67,10 +66,8 @@ public class CommitCompareListFragment extends DialogFragment implements
6766

6867
private DiffStyler diffStyler;
6968

70-
@InjectView(android.R.id.list)
7169
private ListView list;
7270

73-
@InjectView(id.pb_loading)
7471
private ProgressBar progress;
7572

7673
private Repository repository;
@@ -211,6 +208,10 @@ private void addFileStatHeader(List<CommitFile> files,
211208
public void onViewCreated(View view, Bundle savedInstanceState) {
212209
super.onViewCreated(view, savedInstanceState);
213210

211+
ViewFinder finder = new ViewFinder(view);
212+
list = finder.find(android.R.id.list);
213+
progress = finder.find(id.pb_loading);
214+
214215
LayoutInflater inflater = getActivity().getLayoutInflater();
215216

216217
list.setOnItemClickListener(this);

app/src/main/java/com/github/mobile/ui/commit/CommitDiffListFragment.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import com.actionbarsherlock.view.Menu;
4343
import com.actionbarsherlock.view.MenuInflater;
4444
import com.actionbarsherlock.view.MenuItem;
45+
import com.github.kevinsawicki.wishlist.ViewFinder;
4546
import com.github.kevinsawicki.wishlist.ViewUtils;
4647
import com.github.mobile.R.id;
4748
import com.github.mobile.R.layout;
@@ -72,8 +73,6 @@
7273
import org.eclipse.egit.github.core.Repository;
7374
import org.eclipse.egit.github.core.RepositoryCommit;
7475

75-
import roboguice.inject.InjectView;
76-
7776
/**
7877
* Fragment to display commit details with diff output
7978
*/
@@ -82,10 +81,8 @@ public class CommitDiffListFragment extends DialogFragment implements
8281

8382
private DiffStyler diffStyler;
8483

85-
@InjectView(android.R.id.list)
8684
private ListView list;
8785

88-
@InjectView(id.pb_loading)
8986
private ProgressBar progress;
9087

9188
private Repository repository;
@@ -364,6 +361,10 @@ private void updateItems(List<CommitComment> comments,
364361
public void onViewCreated(View view, Bundle savedInstanceState) {
365362
super.onViewCreated(view, savedInstanceState);
366363

364+
ViewFinder finder = new ViewFinder(view);
365+
list = finder.find(android.R.id.list);
366+
progress = finder.find(id.pb_loading);
367+
367368
diffStyler = new DiffStyler(getResources());
368369

369370
list.setOnItemClickListener(this);

app/src/main/java/com/github/mobile/ui/gist/GistFileFragment.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import com.actionbarsherlock.view.Menu;
3232
import com.actionbarsherlock.view.MenuInflater;
3333
import com.actionbarsherlock.view.MenuItem;
34+
import com.github.kevinsawicki.wishlist.ViewFinder;
3435
import com.github.mobile.R.id;
3536
import com.github.mobile.R.layout;
3637
import com.github.mobile.R.menu;
@@ -49,15 +50,12 @@
4950
import org.eclipse.egit.github.core.Gist;
5051
import org.eclipse.egit.github.core.GistFile;
5152

52-
import roboguice.inject.InjectView;
53-
5453
/**
5554
* Fragment to display the content of a file in a Gist
5655
*/
5756
public class GistFileFragment extends DialogFragment implements
5857
OnSharedPreferenceChangeListener {
5958

60-
@InjectView(id.wv_code)
6159
private WebView webView;
6260

6361
private String gistId;
@@ -193,6 +191,9 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
193191
public void onViewCreated(View view, Bundle savedInstanceState) {
194192
super.onViewCreated(view, savedInstanceState);
195193

194+
ViewFinder finder = new ViewFinder(view);
195+
webView = finder.find(id.wv_code);
196+
196197
editor = new SourceEditor(webView);
197198
editor.setWrap(PreferenceUtils.getCodePreferences(getActivity())
198199
.getBoolean(WRAP, false));

app/src/main/java/com/github/mobile/ui/gist/GistFilesViewActivity.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@
4444
import org.eclipse.egit.github.core.Gist;
4545
import org.eclipse.egit.github.core.User;
4646

47-
import roboguice.inject.InjectView;
48-
4947
/**
5048
* Activity to page through the content of all the files in a Gist
5149
*/
@@ -67,13 +65,10 @@ public static Intent createIntent(Gist gist, int position) {
6765

6866
private int initialPosition;
6967

70-
@InjectView(id.vp_pages)
7168
private ViewPager pager;
7269

73-
@InjectView(id.pb_loading)
7470
private ProgressBar loadingBar;
7571

76-
@InjectView(id.tpi_header)
7772
private TitlePageIndicator indicator;
7873

7974
private Gist gist;
@@ -98,6 +93,10 @@ protected void onCreate(Bundle savedInstanceState) {
9893

9994
setContentView(layout.pager_with_title);
10095

96+
pager = finder.find(id.vp_pages);
97+
loadingBar = finder.find(id.pb_loading);
98+
indicator = finder.find(id.tpi_header);
99+
101100
if (initialPosition < 0)
102101
initialPosition = 0;
103102

app/src/main/java/com/github/mobile/ui/gist/GistFragment.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
import com.actionbarsherlock.view.Menu;
3939
import com.actionbarsherlock.view.MenuInflater;
4040
import com.actionbarsherlock.view.MenuItem;
41+
import com.github.kevinsawicki.wishlist.ViewFinder;
4142
import com.github.kevinsawicki.wishlist.ViewUtils;
4243
import com.github.mobile.R.id;
4344
import com.github.mobile.R.layout;
@@ -70,8 +71,6 @@
7071
import org.eclipse.egit.github.core.GistFile;
7172
import org.eclipse.egit.github.core.User;
7273

73-
import roboguice.inject.InjectView;
74-
7574
/**
7675
* Activity to display an existing Gist
7776
*/
@@ -83,10 +82,8 @@ public class GistFragment extends DialogFragment implements OnItemClickListener
8382

8483
private Gist gist;
8584

86-
@InjectView(android.R.id.list)
8785
private ListView list;
8886

89-
@InjectView(id.pb_loading)
9087
private ProgressBar progress;
9188

9289
@Inject
@@ -148,6 +145,10 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
148145
public void onViewCreated(View view, Bundle savedInstanceState) {
149146
super.onViewCreated(view, savedInstanceState);
150147

148+
ViewFinder finder = new ViewFinder(view);
149+
list = finder.find(android.R.id.list);
150+
progress = finder.find(id.pb_loading);
151+
151152
Activity activity = getActivity();
152153
adapter = new HeaderFooterListAdapter<CommentListAdapter>(list,
153154
new CommentListAdapter(activity.getLayoutInflater(), avatars,

app/src/main/java/com/github/mobile/ui/issue/EditIssueActivity.java

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,6 @@
6262
import org.eclipse.egit.github.core.service.LabelService;
6363
import org.eclipse.egit.github.core.service.MilestoneService;
6464

65-
import roboguice.inject.InjectView;
66-
6765
/**
6866
* Activity to edit or create an issue
6967
*/
@@ -102,28 +100,20 @@ public static Intent createIntent(final Issue issue,
102100
return builder.toIntent();
103101
}
104102

105-
@InjectView(id.et_issue_title)
106103
private EditText titleText;
107104

108-
@InjectView(id.et_issue_body)
109105
private EditText bodyText;
110106

111-
@InjectView(id.ll_milestone_graph)
112107
private View milestoneGraph;
113108

114-
@InjectView(id.tv_milestone)
115109
private TextView milestoneText;
116110

117-
@InjectView(id.v_closed)
118111
private View milestoneClosed;
119112

120-
@InjectView(id.iv_assignee_avatar)
121113
private ImageView assigneeAvatar;
122114

123-
@InjectView(id.tv_assignee_name)
124115
private TextView assigneeText;
125116

126-
@InjectView(id.tv_labels)
127117
private TextView labelsText;
128118

129119
@Inject
@@ -156,6 +146,15 @@ protected void onCreate(Bundle savedInstanceState) {
156146

157147
setContentView(layout.issue_edit);
158148

149+
titleText = finder.find(id.et_issue_title);
150+
bodyText = finder.find(id.et_issue_body);
151+
milestoneGraph = finder.find(id.ll_milestone_graph);
152+
milestoneText = finder.find(id.tv_milestone);
153+
milestoneClosed = finder.find(id.v_closed);
154+
assigneeAvatar = finder.find(id.iv_assignee_avatar);
155+
assigneeText = finder.find(id.tv_assignee_name);
156+
labelsText = finder.find(id.tv_labels);
157+
159158
Intent intent = getIntent();
160159

161160
if (savedInstanceState != null)

app/src/main/java/com/github/mobile/ui/issue/EditIssuesFilterActivity.java

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@
5050
import org.eclipse.egit.github.core.service.LabelService;
5151
import org.eclipse.egit.github.core.service.MilestoneService;
5252

53-
import roboguice.inject.InjectView;
54-
5553
/**
5654
* Activity to create or edit an issues filter for a repository
5755
*/
@@ -94,16 +92,12 @@ public static Intent createIntent(IssueFilter filter) {
9492

9593
private IssueFilter filter;
9694

97-
@InjectView(id.tv_labels)
9895
private TextView labelsText;
9996

100-
@InjectView(id.tv_milestone)
10197
private TextView milestoneText;
10298

103-
@InjectView(id.tv_assignee)
10499
private TextView assigneeText;
105100

106-
@InjectView(id.iv_avatar)
107101
private ImageView avatarView;
108102

109103
@Override
@@ -112,6 +106,11 @@ protected void onCreate(Bundle savedInstanceState) {
112106

113107
setContentView(layout.issues_filter_edit);
114108

109+
labelsText = finder.find(id.tv_labels);
110+
milestoneText = finder.find(id.tv_milestone);
111+
assigneeText = finder.find(id.tv_assignee);
112+
avatarView = finder.find(id.iv_avatar);
113+
115114
if (savedInstanceState != null)
116115
filter = (IssueFilter) savedInstanceState
117116
.getSerializable(EXTRA_ISSUE_FILTER);

app/src/main/java/com/github/mobile/ui/issue/IssueFragment.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
import com.actionbarsherlock.view.Menu;
5151
import com.actionbarsherlock.view.MenuInflater;
5252
import com.actionbarsherlock.view.MenuItem;
53+
import com.github.kevinsawicki.wishlist.ViewFinder;
5354
import com.github.kevinsawicki.wishlist.ViewUtils;
5455
import com.github.mobile.R.drawable;
5556
import com.github.mobile.R.id;
@@ -83,8 +84,6 @@
8384
import org.eclipse.egit.github.core.RepositoryId;
8485
import org.eclipse.egit.github.core.User;
8586

86-
import roboguice.inject.InjectView;
87-
8887
/**
8988
* Fragment to display an issue
9089
*/
@@ -106,10 +105,8 @@ public class IssueFragment extends DialogFragment {
106105
@Inject
107106
private IssueStore store;
108107

109-
@InjectView(android.R.id.list)
110108
private ListView list;
111109

112-
@InjectView(id.pb_loading)
113110
private ProgressBar progress;
114111

115112
private View headerView;
@@ -255,6 +252,10 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
255252
public void onViewCreated(View view, Bundle savedInstanceState) {
256253
super.onViewCreated(view, savedInstanceState);
257254

255+
ViewFinder finder = new ViewFinder(view);
256+
list = finder.find(android.R.id.list);
257+
progress = finder.find(id.pb_loading);
258+
258259
LayoutInflater inflater = getLayoutInflater(savedInstanceState);
259260

260261
headerView = inflater.inflate(layout.issue_header, null);

app/src/main/java/com/github/mobile/ui/issue/IssuesViewActivity.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,6 @@
4848
import org.eclipse.egit.github.core.RepositoryIssue;
4949
import org.eclipse.egit.github.core.User;
5050

51-
import roboguice.inject.InjectView;
52-
5351
/**
5452
* Activity to display a collection of issues or pull requests in a pager
5553
*/
@@ -146,7 +144,6 @@ public static Intent createIntent(Collection<? extends Issue> issues,
146144
return builder.toIntent();
147145
}
148146

149-
@InjectView(id.vp_pages)
150147
private ViewPager pager;
151148

152149
private int[] issueNumbers;
@@ -183,6 +180,8 @@ protected void onCreate(Bundle savedInstanceState) {
183180

184181
setContentView(layout.pager);
185182

183+
pager = finder.find(id.vp_pages);
184+
186185
if (repo != null)
187186
adapter = new IssuesPagerAdapter(this, repo, issueNumbers);
188187
else

0 commit comments

Comments
 (0)