Skip to content

Commit 30c6f11

Browse files
authored
Merge pull request #88 from huttneab/phone_bug_fix
fix composite subscription bug
2 parents fbde7ef + 232f294 commit 30c6f11

5 files changed

Lines changed: 10 additions & 7 deletions

File tree

app/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Manifest version information!
22
def versionMajor = 1
33
def versionMinor = 1
4-
def versionPatch = 0
4+
def versionPatch = 1
55
def versionBuild = 0 // bump for dogfood builds, public betas, etc.
66

77
apply plugin: 'com.android.application'

app/src/main/java/com/rockthevote/grommet/ui/registration/AdditionalInfoFragment.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ public class AdditionalInfoFragment extends BaseRegistrationFragment {
103103
@Inject BriteDatabase db;
104104

105105
private ObservableValidator validator;
106-
private CompositeSubscription subscriptions = new CompositeSubscription();
106+
private CompositeSubscription subscriptions;
107107
private EnumAdapter<Race> raceEnumAdapter;
108108
private EnumAdapter<Party> partyEnumAdapter;
109109

@@ -192,9 +192,9 @@ public void onActivityCreated(@Nullable Bundle savedInstanceState) {
192192
@Override
193193
public void onResume() {
194194
super.onResume();
195-
196195
phoneFormatter = new PhoneNumberFormattingTextWatcher();
197196
phone.addTextChangedListener(phoneFormatter);
197+
subscriptions = new CompositeSubscription();
198198

199199
subscriptions.add(RxTextView.afterTextChangeEvents(raceSpinner.getEditText())
200200
.observeOn(Schedulers.io())
@@ -333,8 +333,8 @@ public void onResume() {
333333
@Override
334334
public void onPause() {
335335
super.onPause();
336-
subscriptions.unsubscribe();
337336
phone.removeTextChangedListener(phoneFormatter);
337+
subscriptions.unsubscribe();
338338
}
339339

340340
/**

app/src/main/java/com/rockthevote/grommet/ui/registration/AssistantInfoFragment.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public class AssistantInfoFragment extends BaseRegistrationFragment {
6464

6565
private PhoneNumberFormattingTextWatcher phoneFormatter;
6666

67-
private CompositeSubscription subscriptions = new CompositeSubscription();
67+
private CompositeSubscription subscriptions;
6868

6969
@Nullable
7070
@Override
@@ -89,6 +89,7 @@ public void onResume() {
8989
phoneFormatter = new PhoneNumberFormattingTextWatcher();
9090
phoneEditText.addTextChangedListener(phoneFormatter);
9191

92+
subscriptions = new CompositeSubscription();
9293
subscriptions.add(RxTextView.afterTextChangeEvents(phoneEditText)
9394
.observeOn(Schedulers.io())
9495
.debounce(DEBOUNCE, TimeUnit.MILLISECONDS)

app/src/main/java/com/rockthevote/grommet/ui/views/AddressView.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ public class AddressView extends FrameLayout {
8181
private ArrayAdapter<CharSequence> stateAdapter;
8282

8383
private Address.Type type;
84-
private CompositeSubscription subscriptions = new CompositeSubscription();
84+
private CompositeSubscription subscriptions;
8585
private ZipTextWatcher zipTextWatcher = new ZipTextWatcher();
8686

8787
public AddressView(Context context) {
@@ -192,6 +192,7 @@ protected void onAttachedToWindow() {
192192
if (!isInEditMode()) {
193193
zipEditText.addTextChangedListener(zipTextWatcher);
194194

195+
subscriptions = new CompositeSubscription();
195196
subscriptions.add(Observable.combineLatest(RxTextView.afterTextChangeEvents(streetEditText),
196197
RxTextView.afterTextChangeEvents(unitEditText),
197198
RxTextView.afterTextChangeEvents(cityEditText),

app/src/main/java/com/rockthevote/grommet/ui/views/NameView.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public class NameView extends FrameLayout {
7373

7474
private Name.Type type;
7575

76-
private CompositeSubscription subscriptions = new CompositeSubscription();
76+
private CompositeSubscription subscriptions;
7777

7878
public NameView(Context context) {
7979
this(context, null);
@@ -156,6 +156,7 @@ protected void onFinishInflate() {
156156
protected void onAttachedToWindow() {
157157
super.onAttachedToWindow();
158158
if (!isInEditMode()) {
159+
subscriptions = new CompositeSubscription();
159160
subscriptions.add(Observable.combineLatest(RxTextView.afterTextChangeEvents(firstNameEditText),
160161
RxTextView.afterTextChangeEvents(middleNameEditText),
161162
RxTextView.afterTextChangeEvents(lastNameEditText),

0 commit comments

Comments
 (0)