Skip to content

Commit a5cff54

Browse files
authored
Merge pull request #192 from OceanLabs/address-fix
Address fix
2 parents 42976b5 + 0a824e2 commit a5cff54

File tree

13 files changed

+52
-6
lines changed

13 files changed

+52
-6
lines changed

KitePrintSDK/CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
### 5.8.4
2+
- Added: Check for address required fields and alert message when trying to proceed to checkout without filling in an address
3+
14
### 5.8.3
25
- Fixed: Previous versions of the sdk that stored unencrypted data should now be compatible with current versions of the sdk and encrypt the existing data
36

KitePrintSDK/src/main/java/ly/kite/KiteSDK.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ public class KiteSDK
110110
static public final boolean DISPLAY_PRODUCTS = false;
111111

112112

113-
static public final String SDK_VERSION = "5.8.3";
113+
static public final String SDK_VERSION = "5.8.4";
114114

115115
static public final String IMAGE_CATEGORY_APP = "app";
116116
static public final String IMAGE_CATEGORY_PRODUCT_ITEM = "product_item";

KitePrintSDK/src/main/java/ly/kite/address/Address.java

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -416,6 +416,26 @@ public boolean equals( Object otherObject )
416416
Country.areBothNullOrEqual( mCountry, otherAddress.mCountry ) );
417417
}
418418

419+
public boolean isFilledIn() {
420+
if(mRecipientName == null || mRecipientName.length() < 1) {
421+
return false;
422+
}
423+
424+
if(mLine1 == null || mLine1.length() < 1) {
425+
return false;
426+
}
427+
428+
if(mCity == null || mCity.length() < 1) {
429+
return false;
430+
}
431+
432+
if(mZIPOrPostalCode == null || mZIPOrPostalCode.length() < 1) {
433+
return false;
434+
}
435+
436+
return true;
437+
}
438+
419439

420440
////////// Inner Class(es) //////////
421441

KitePrintSDK/src/main/java/ly/kite/address/AddressEditActivity.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -296,6 +296,15 @@ public void onSaveClicked()
296296
return;
297297
}
298298

299+
// Get and verify the city
300+
301+
String city = mAddressCityEditText.getText().toString();
302+
303+
if ( city.trim().length() == 0 )
304+
{
305+
displayModalDialog( R.string.alert_dialog_title_oops, R.string.alert_dialog_message_no_city, R.string.OK, null, 0, null );
306+
return;
307+
}
299308

300309
// Get and verify the post code
301310

KitePrintSDK/src/main/java/ly/kite/checkout/ShippingActivity.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,11 @@ public void onClick( View view )
322322
*****************************************************/
323323
private void onShippingAddress( Address shippingAddress )
324324
{
325-
mAddressPickerButton.setText( shippingAddress.toMultiLineText() );
325+
if(shippingAddress != null && shippingAddress.isFilledIn()) {
326+
mAddressPickerButton.setText(shippingAddress.toMultiLineText());
327+
} else {
328+
mAddressPickerButton.setText( getString(R.string.shipping_delivery_address_button_text));
329+
}
326330
}
327331

328332

KitePrintSDK/src/main/java/ly/kite/journey/basket/BasketActivity.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ protected void onRightClicked()
423423

424424
// Make sure we have a shipping address
425425

426-
if ( mShippingAddress == null )
426+
if ( mShippingAddress == null || !mShippingAddress.isFilledIn())
427427
{
428428
showErrorDialog( R.string.alert_dialog_title_invalid_delivery_address, R.string.alert_dialog_message_invalid_delivery_address );
429429

@@ -697,7 +697,11 @@ private void onNewShippingDetails()
697697
*****************************************************/
698698
private void onShippingAddress()
699699
{
700-
mDeliveryAddressTextView.setText( mShippingAddress.toSingleLineText() );
700+
if(mShippingAddress != null && mShippingAddress.isFilledIn()) {
701+
mDeliveryAddressTextView.setText(mShippingAddress.toSingleLineText());
702+
} else {
703+
mDeliveryAddressTextView.setText(getString(R.string.shipping_delivery_address_button_text));
704+
}
701705

702706
checkRequestPrices();
703707
}

KitePrintSDK/src/main/res/values-da/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@
8888
<string name="alert_dialog_message_no_line1">Udfyld venligst linje 1 i adressen</string>
8989
<string name="alert_dialog_message_no_postal_code">Udfyld venligst dit postnummer / ZIP kode</string>
9090
<string name="alert_dialog_message_no_email_address">Indtast venligst en e-mail adresse</string>
91+
<string name="alert_dialog_message_no_city">Udfyld venligst din by</string>
9192

9293
<string name="alert_dialog_title_invalid_delivery_address">Ugyldig leveringsadresse</string>
9394
<string name="alert_dialog_message_invalid_delivery_address">Vælg venligst en leveringsadresse</string>

KitePrintSDK/src/main/res/values-de/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@
9090
<string name="alert_dialog_message_no_line1">Bitte fülle Zeile 1 der Adresse aus.</string>
9191
<string name="alert_dialog_message_no_postal_code">Bitte gib die Postleitzahl ein.</string>
9292
<string name="alert_dialog_message_no_email_address">Bitte gib deine E-Mail-Adresse ein.</string>
93+
<string name="alert_dialog_message_no_city">Bitte geben Du Ihre Stadt.</string>
9394

9495
<string name="alert_dialog_title_invalid_delivery_address">Ungültige Lieferadresse</string>
9596
<string name="alert_dialog_message_invalid_delivery_address">Bitte eine Lieferadresse auswählen.</string>

KitePrintSDK/src/main/res/values-es/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@
9494
<string name="alert_dialog_message_no_line1">Por favor, rellene la línea 1 de la dirección</string>
9595
<string name="alert_dialog_message_no_postal_code">Introduzca su código postal</string>
9696
<string name="alert_dialog_message_no_email_address">Por favor introduzca un correo eléctronico</string>
97+
<string name="alert_dialog_message_no_city">Por favor, ingrese su ciudad</string>
9798

9899
<string name="alert_dialog_title_invalid_delivery_address">Dirección de entrega no válida</string>
99100
<string name="alert_dialog_message_invalid_delivery_address">Por favor, elija una dirección de entrega</string>

KitePrintSDK/src/main/res/values-fr/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@
9191
<string name="alert_dialog_message_no_line1">Veuillez remplir la ligne 1 de l\'adresse</string>
9292
<string name="alert_dialog_message_no_postal_code">Veuillez remplir votre code postal</string>
9393
<string name="alert_dialog_message_no_email_address">Veuillez remplir votre adresse email</string>
94+
<string name="alert_dialog_message_no_city">Merci de remplir votre ville</string>
9495

9596
<string name="alert_dialog_title_invalid_delivery_address">Adresse de livraison invalide</string>
9697
<string name="alert_dialog_message_invalid_delivery_address">S\'il vous plaît choisir une adresse de livraison</string>

0 commit comments

Comments
 (0)