Skip to content

Commit 4f73c5d

Browse files
liveHarshitnikit19
authored andcommitted
fix: Progress in Attendee fragment (#1872) (#1876)
1 parent 4205fbc commit 4f73c5d

File tree

3 files changed

+32
-31
lines changed

3 files changed

+32
-31
lines changed

app/src/main/java/org/fossasia/openevent/general/attendees/AttendeeFragment.kt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,10 @@ class AttendeeFragment : Fragment() {
156156
.observe(viewLifecycleOwner, Observer {
157157
rootView.progressBarAttendee.isVisible = it
158158
rootView.register.isEnabled = !it
159+
rootView.register.text = if (!it) getString(R.string.register) else ""
160+
rootView.register.backgroundTintList =
161+
if (it) resources.getColorStateList(R.color.grey)
162+
else resources.getColorStateList(R.color.colorAccent)
159163
})
160164

161165
rootView.sameBuyerCheckBox.setOnCheckedChangeListener { buttonView, isChecked ->

app/src/main/java/org/fossasia/openevent/general/attendees/AttendeeViewModel.kt

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -132,10 +132,6 @@ class AttendeeViewModel(
132132
.withDefaultSchedulers()
133133
.doOnSubscribe {
134134
mutableProgress.value = true
135-
}.doFinally {
136-
createAttendeeIterations++
137-
if (createAttendeeIterations == totalAttendee)
138-
mutableProgress.value = false
139135
}.subscribe({
140136
attendeesForOrder.add(it)
141137
if (attendeesForOrder.size == totalAttendee) {
@@ -233,11 +229,7 @@ class AttendeeViewModel(
233229
}
234230
compositeDisposable += orderService.placeOrder(order)
235231
.withDefaultSchedulers()
236-
.doOnSubscribe {
237-
mutableProgress.value = true
238-
}.doFinally {
239-
mutableProgress.value = false
240-
}.subscribe({
232+
.subscribe({
241233
orderIdentifier = it.identifier.toString()
242234
Timber.d("Success placing order!")
243235
when (it.paymentMode) {
@@ -264,9 +256,7 @@ class AttendeeViewModel(
264256
private fun confirmOrderStatus(identifier: String, order: ConfirmOrder) {
265257
compositeDisposable += orderService.confirmOrder(identifier, order)
266258
.withDefaultSchedulers()
267-
.doOnSubscribe {
268-
mutableProgress.value = true
269-
}.doFinally {
259+
.doFinally {
270260
mutableProgress.value = false
271261
}.subscribe({
272262
mutableMessage.value = resource.getString(R.string.order_success_message)

app/src/main/res/layout/fragment_attendee.xml

Lines changed: 26 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -585,28 +585,35 @@
585585
android:layout_height="wrap_content" />
586586
</LinearLayout>
587587

588-
<com.google.android.material.button.MaterialButton
589-
android:id="@+id/register"
590-
style="@style/AppTheme.MaterialButton.RoundedCorners"
588+
<RelativeLayout
591589
android:layout_width="match_parent"
592590
android:layout_height="wrap_content"
593-
android:layout_gravity="center"
594-
android:layout_marginStart="@dimen/details_margin_small"
595-
android:layout_marginEnd="@dimen/details_margin_small"
596-
android:layout_marginTop="@dimen/layout_margin_large"
597-
android:layout_marginBottom="@dimen/layout_margin_large"
598-
android:enabled="false"
599-
android:text="@string/register"/>
600-
</LinearLayout>
591+
android:layout_gravity="center">
601592

602-
<ProgressBar
603-
android:id="@+id/progressBarAttendee"
604-
android:layout_width="wrap_content"
605-
android:layout_height="wrap_content"
606-
android:layout_gravity="center"
607-
android:elevation="@dimen/card_elevation"
608-
android:padding="@dimen/padding_small"
609-
android:visibility="gone" />
593+
<com.google.android.material.button.MaterialButton
594+
android:id="@+id/register"
595+
style="@style/AppTheme.MaterialButton.RoundedCorners"
596+
android:layout_width="match_parent"
597+
android:layout_height="wrap_content"
598+
android:layout_marginStart="@dimen/details_margin_small"
599+
android:layout_marginEnd="@dimen/details_margin_small"
600+
android:layout_marginTop="@dimen/layout_margin_large"
601+
android:layout_marginBottom="@dimen/layout_margin_large"
602+
android:enabled="false"
603+
android:text="@string/register"/>
604+
605+
<ProgressBar
606+
android:layout_centerInParent="true"
607+
android:id="@+id/progressBarAttendee"
608+
android:layout_width="@dimen/spinner_height"
609+
android:layout_height="@dimen/spinner_width"
610+
android:elevation="@dimen/card_elevation"
611+
android:padding="@dimen/padding_small"
612+
android:visibility="gone"
613+
tools:visibility="visible"/>
614+
</RelativeLayout>
615+
616+
</LinearLayout>
610617
</LinearLayout>
611618
</androidx.coordinatorlayout.widget.CoordinatorLayout>
612619
</androidx.core.widget.NestedScrollView>

0 commit comments

Comments
 (0)