From c57b171076724e068b3de2ddeb98692323edfe31 Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Fri, 16 Sep 2016 11:51:44 +0200 Subject: [PATCH 01/16] Prevent context leak in isConnectedToNetwork Fixes #177 --- hockeysdk/src/main/java/net/hockeyapp/android/utils/Util.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/utils/Util.java b/hockeysdk/src/main/java/net/hockeyapp/android/utils/Util.java index f15a6d1d2..4ae4785f4 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/utils/Util.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/utils/Util.java @@ -264,7 +264,7 @@ private static Bundle getBundle(Context context) { } public static boolean isConnectedToNetwork(Context context) { - ConnectivityManager connectivityManager = (ConnectivityManager) context.getSystemService(Context.CONNECTIVITY_SERVICE); + ConnectivityManager connectivityManager = (ConnectivityManager) context.getApplicationContext().getSystemService(Context.CONNECTIVITY_SERVICE); if (connectivityManager != null) { NetworkInfo activeNetwork = connectivityManager.getActiveNetworkInfo(); return activeNetwork != null && activeNetwork.isConnected(); From 6564ca9563eb8559abd9a9b9e6672403bce7c262 Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Fri, 16 Sep 2016 12:09:11 +0200 Subject: [PATCH 02/16] Fix issue with feedback UI not being hidden --- .../src/main/java/net/hockeyapp/android/FeedbackActivity.java | 3 +++ .../src/main/java/net/hockeyapp/android/FeedbackManager.java | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java index 85df43f6b..a07d763ce 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java @@ -518,6 +518,9 @@ protected void configureFeedbackView(boolean haveToken) { mFeedbackViewInitialized = true; } + mNameInput.setVisibility(FeedbackManager.getRequireUserName() == FeedbackUserDataElement.DONT_SHOW ? View.GONE : View.VISIBLE); + mEmailInput.setVisibility(FeedbackManager.getRequireUserEmail() == FeedbackUserDataElement.DONT_SHOW ? View.GONE : View.VISIBLE); + /** Reset the remaining fields if previously populated */ mTextInput.setText(""); diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackManager.java b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackManager.java index c7aa8b657..070648400 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackManager.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackManager.java @@ -81,12 +81,12 @@ public class FeedbackManager { /** * Whether the user's name is required. */ - private static FeedbackUserDataElement requireUserName; + private static FeedbackUserDataElement requireUserName = FeedbackUserDataElement.REQUIRED; /** * Whether the user's email is required. */ - private static FeedbackUserDataElement requireUserEmail; + private static FeedbackUserDataElement requireUserEmail = FeedbackUserDataElement.REQUIRED; private static String userName; From 312751762f9cf3531651edee9c61c3e5a0aed6c1 Mon Sep 17 00:00:00 2001 From: Jared Burrows Date: Thu, 20 Oct 2016 23:04:10 -0700 Subject: [PATCH 03/16] fix npe on listfiles in isFreeSpaceAvailable() method --- .../main/java/net/hockeyapp/android/metrics/Persistence.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/metrics/Persistence.java b/hockeysdk/src/main/java/net/hockeyapp/android/metrics/Persistence.java index 492296cec..4fa504f50 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/metrics/Persistence.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/metrics/Persistence.java @@ -269,12 +269,13 @@ protected boolean isFreeSpaceAvailable() { // TODO Check for available disk space as well. synchronized (LOCK) { Context context = getContext(); - if ((context.getFilesDir()) != null) { + if (context.getFilesDir() != null) { File filesDir = context.getFilesDir(); String path = filesDir.getAbsolutePath() + BIT_TELEMETRY_DIRECTORY; if (!TextUtils.isEmpty(path)) { File dir = new File(path); - return (dir.listFiles().length < MAX_FILE_COUNT); + File[] files = dir.listFiles(); + return files != null && files.length < MAX_FILE_COUNT; } } return false; From fedc26a97ae1f0ec825b87bf6709cb197ebd7a14 Mon Sep 17 00:00:00 2001 From: Jared Burrows Date: Thu, 20 Oct 2016 23:23:28 -0700 Subject: [PATCH 04/16] fix npe on equals() in FeedbackActivity --- .../src/main/java/net/hockeyapp/android/FeedbackActivity.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java index a07d763ce..3a7a52c15 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/FeedbackActivity.java @@ -794,10 +794,10 @@ public void handleMessage(Message msg) { String responseString = bundle.getString(SendFeedbackTask.BUNDLE_FEEDBACK_RESPONSE); String statusCode = bundle.getString(SendFeedbackTask.BUNDLE_FEEDBACK_STATUS); String requestType = bundle.getString(SendFeedbackTask.BUNDLE_REQUEST_TYPE); - if ((requestType.equals("send") && ((responseString == null) || (Integer.parseInt(statusCode) != 201)))) { + if ("send".equals(requestType) && (responseString == null || Integer.parseInt(statusCode) != 201)) { // Send feedback went wrong if response is empty or status code != 201 error.setMessage(feedbackActivity.getString(R.string.hockeyapp_feedback_send_generic_error)); - } else if ((requestType.equals("fetch") && (statusCode != null) && ((Integer.parseInt(statusCode) == 404) || (Integer.parseInt(statusCode) == 422)))) { + } else if ("fetch".equals(requestType) && statusCode != null && (Integer.parseInt(statusCode) == 404 || Integer.parseInt(statusCode) == 422)) { // Fetch feedback went wrong if status code is 404 or 422 feedbackActivity.resetFeedbackView(); success = true; From 0ace6648b2b1bd954d47f3d125457d22ca6b360a Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Mon, 24 Oct 2016 13:18:31 +0200 Subject: [PATCH 05/16] Safe-guard device resolution against NPE MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit NPEs occurred due to WindowManager.getDefaultDisplay() returnin `null` which shouldn’t happen but actually did. --- .../android/metrics/TelemetryContext.java | 25 ++++++++++++++----- 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/metrics/TelemetryContext.java b/hockeysdk/src/main/java/net/hockeyapp/android/metrics/TelemetryContext.java index 29081a8ab..5d9bc027e 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/metrics/TelemetryContext.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/metrics/TelemetryContext.java @@ -235,9 +235,16 @@ protected void updateScreenResolution() { Context.WINDOW_SERVICE); if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN_MR1) { Point size = new Point(); - wm.getDefaultDisplay().getRealSize(size); - width = size.x; - height = size.y; + Display d = wm.getDefaultDisplay(); + if (d != null) { + d.getRealSize(size); + width = size.x; + height = size.y; + } else { + width = 0; + height = 0; + } + } else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB_MR2) { try { //We have to use undocumented API here. Android 4.0 introduced soft buttons for @@ -250,9 +257,15 @@ protected void updateScreenResolution() { height = (Integer) mGetRawH.invoke(display); } catch (Exception ex) { Point size = new Point(); - wm.getDefaultDisplay().getSize(size); - width = size.x; - height = size.y; + Display d = wm.getDefaultDisplay(); + if (d != null) { + d.getRealSize(size); + width = size.x; + height = size.y; + } else { + width = 0; + height = 0; + } HockeyLog.debug(TAG, "Couldn't determine screen resolution: " + ex.toString()); } From 9ad786cb7ecb61ee2c9ce6bc11a1b1b69e5cb64f Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Mon, 14 Nov 2016 18:20:09 -0800 Subject: [PATCH 06/16] Correctly notify listener when dialog cancelled Fixes #168 --- .../hockeyapp/android/tasks/CheckUpdateTaskWithUI.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/hockeysdk/src/main/java/net/hockeyapp/android/tasks/CheckUpdateTaskWithUI.java b/hockeysdk/src/main/java/net/hockeyapp/android/tasks/CheckUpdateTaskWithUI.java index 162b4490a..eeab39858 100644 --- a/hockeysdk/src/main/java/net/hockeyapp/android/tasks/CheckUpdateTaskWithUI.java +++ b/hockeysdk/src/main/java/net/hockeyapp/android/tasks/CheckUpdateTaskWithUI.java @@ -96,6 +96,16 @@ public void onClick(DialogInterface dialog, int which) { } }); + builder.setOnCancelListener(new DialogInterface.OnCancelListener() { + @Override + public void onCancel(DialogInterface dialog) { + cleanUp(); + if (null != listener) { + listener.onCancel(); + } + } + }); + builder.setPositiveButton(R.string.hockeyapp_update_dialog_positive_button, new DialogInterface.OnClickListener() { public void onClick(DialogInterface dialog, int which) { if (getCachingEnabled()) { From 4d0b4c4cefc9908fe457acd02c8e1f0b2926f3a9 Mon Sep 17 00:00:00 2001 From: Thomas Dohmke Date: Fri, 18 Nov 2016 09:27:34 -0800 Subject: [PATCH 07/16] Fix typo --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 9c1eccbf8..68b1de484 100644 --- a/README.md +++ b/README.md @@ -231,7 +231,7 @@ This will add the ability for your users to provide feedback from right inside y ```java import net.hockeyapp.android.FeedbackManager; -public class YourActivity extends Activitiy { +public class YourActivity extends Activity { @Override public void onCreate(Bundle savedInstanceState) { From 65c7717202b55b1bcbbd49b8a220604d6e11083c Mon Sep 17 00:00:00 2001 From: Giancarlo Romeo Date: Mon, 21 Nov 2016 16:57:26 +0100 Subject: [PATCH 08/16] Add italian localization (#170) * Added italian localization --- hockeysdk/src/main/res/values-it/strings.xml | 94 ++++++++++++++++++++ 1 file changed, 94 insertions(+) create mode 100644 hockeysdk/src/main/res/values-it/strings.xml diff --git a/hockeysdk/src/main/res/values-it/strings.xml b/hockeysdk/src/main/res/values-it/strings.xml new file mode 100644 index 000000000..9833bb43a --- /dev/null +++ b/hockeysdk/src/main/res/values-it/strings.xml @@ -0,0 +1,94 @@ + + + Questa app + %s chiusa inaspettatamente + Vuoi inviare un report in modo anonimo così possiamo risolvere il problema? + Non inviare + Invia sempre + Invia report + + + Download fallito + Impossibile scaricare l\'aggiornamento. Vuoi riprovare? + Annulla + Riprova + + + %s è dispobinile ed è un aggiornamento obbligatorio! + Aggiornamento disponibile + Aggiorna + Mostrare informazioni sul nuovo aggiornamento? + Ignora + Mostra + + + Build scaduta + %s è scaduta. Controlla HockeyApp per aggiornamenti. + + + In attesa + Vuoi mandare il tuo feedback nuovamente? + Nome + E-mail + Soggetto + Messaggio + Ultimo aggiornamento: %s + Aggiungi allegato + Invia feedback + Aggiungi una risposta + Aggiorna + Feedback + Impossibile pubblicare il messaggio. Controlla i valori inseriti e la tua connessione, quindi prova di nuovo. + Nessuna risposta dal server. Controlla la tua connessione, quindi prova di nuovo. + Inserisci un soggetto + Inserisci un indirizzo e-mail valido + Inserisci un indirizzo e-mail + Inserisci il tuo nome + Impossibile pubblicare il messaggio. Inserisci un testo per il feedback + Si è verificato un errore + Allega un file + Allega un\'immagine + Seleziona il file + Seleziona l\'immagine + E\' consentito un massimo di %s allegati. + Caricamento… + Errore + + + Invio feedback… + Recupero discussioni… + + + Inserisci l\'indirizzo e-mail e la password del tuo account HockeyApp per autorizzare l\'accesso a questa app. + Inserisci l\'indirizzo e-mail del tuo account HockeyApp per autorizzare l\'accesso a questa app. + Inserisci le credenziali dell\'account mancanti. + E-mail + Password + Accedi + + + Disegna qualcosa! + Salva + Annulla + Cancella + Vuoi salvare i tuoi disegni? + Scarta + Salva + Annulla + + + Accesso allo spazio di archiviazione richiesto + Per scaricare ed installare gli aggiornamenti dell\'app devi consentire l\'accesso allo spazio di archiviazione del dispositivo. + Annulla + Riprova + + + OK + Annulla + Errore + Si è verificato un errore + + + Il tuo dispositivo non è connesso ad internet. Risolvi i problemi di connessione e riprova. + + From 52fcce9276bf582ed3978cbd607949a1baff30c9 Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Mon, 21 Nov 2016 16:58:42 +0100 Subject: [PATCH 09/16] Fix typo in Italian localization --- hockeysdk/src/main/res/values-it/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hockeysdk/src/main/res/values-it/strings.xml b/hockeysdk/src/main/res/values-it/strings.xml index 9833bb43a..8f9313f1a 100644 --- a/hockeysdk/src/main/res/values-it/strings.xml +++ b/hockeysdk/src/main/res/values-it/strings.xml @@ -14,7 +14,7 @@ Riprova - %s è dispobinile ed è un aggiornamento obbligatorio! + %s è disponibile ed è un aggiornamento obbligatorio! Aggiornamento disponibile Aggiorna Mostrare informazioni sul nuovo aggiornamento? From 979276ae3a85017253c46f3d0541adb3eae5e070 Mon Sep 17 00:00:00 2001 From: Rafael Roman Date: Mon, 21 Nov 2016 17:01:43 +0100 Subject: [PATCH 10/16] pt-br hockeyapp translation (#159) --- hockeysdk/src/main/res/values-pt/strings.xml | 95 ++++++++++++++++++++ 1 file changed, 95 insertions(+) create mode 100644 hockeysdk/src/main/res/values-pt/strings.xml diff --git a/hockeysdk/src/main/res/values-pt/strings.xml b/hockeysdk/src/main/res/values-pt/strings.xml new file mode 100644 index 000000000..bc75b1699 --- /dev/null +++ b/hockeysdk/src/main/res/values-pt/strings.xml @@ -0,0 +1,95 @@ + + + Esta App + %s Parou inesperadamente + Gostaria de enviar um relatório anônimo para corrigirmos o problema? + Não enviar + Sempre enviar + Enviar relatório + + + Falha no download + Não foi possivel baixar a atualização. Gostaria de tentar de novo? + Cancelar + Tentar novamente + + + %s está disponível e é obrigatória! + Atualização disponível + Atualizar + Mostrar informações sobre a nova atualização? + Ignorar + Mostrar + + + + Build Expirada + %s expirou. Por favor, verifique no HockeyApp para atualizações. + + + Pendente + Gostaria de enviar o feedback novamente? + Nome + Email + Assunto + Mensagem + Ultima atualização: %s + Adicionar anexo + Enviar feedback + Adicionar Resposta + Atualizar + Feedback + A mensagem não pôde ser enviada. Por favor verifique os campos, sua conexão com a internet e tente novamente.. + Sem resposta do servidor. Por favor, verifique sua conexão e tente novamente. + Por favor preencha um assunto + Por favor preencha seu nome + Por favor preencha seu email + Por favor preencha a mensagem + A mensagem não pôde ser enviada. Por favor verifique o formato do seu email. + Ocorreu um erro + Anexar arquivo + Anexar imagem + Selecionar arquivo + Selecionar imagem + Máximo %s anexos permitidos. + Carregando… + Erro + + + Enviando Feedback… + Recebendo discussões… + + + Por favor preencha seu email e senha do HockeyApp para ter acesso a esse aplicativo. + Por favor preencha seu email do HockeyApp para ter acesso a esse aplicativo. + Por favor preencha suas credenciais. + Email + Senha + Login + + + Desenhe algo! + Salvar + Desfazer + Limpar + Quer salvar seu desenho? + Descartar + Salvar + Cancelar + + + Acesso ao armazenamento necessário + Para poder fazer download e instalar atualizações, você deve permitir que esse aplicativo acesse o armazenamento do seu dispositivo. + Cancelar + Tentar novamente + + + OK + Cancelar + Erro + Ocorreu um erro + + + Seu dispositivo não está conectado à internet. Por favor verifique sua conexão e tente novamente. + + From 6343e151f8edcca69e0186e5ee38748a13364055 Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Mon, 21 Nov 2016 17:02:41 +0100 Subject: [PATCH 11/16] Fix typo in Brazilian-Portuguese translation --- hockeysdk/src/main/res/values-pt/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hockeysdk/src/main/res/values-pt/strings.xml b/hockeysdk/src/main/res/values-pt/strings.xml index bc75b1699..1fc6f57aa 100644 --- a/hockeysdk/src/main/res/values-pt/strings.xml +++ b/hockeysdk/src/main/res/values-pt/strings.xml @@ -39,7 +39,7 @@ Adicionar Resposta Atualizar Feedback - A mensagem não pôde ser enviada. Por favor verifique os campos, sua conexão com a internet e tente novamente.. + A mensagem não pôde ser enviada. Por favor verifique os campos, sua conexão com a internet e tente novamente… Sem resposta do servidor. Por favor, verifique sua conexão e tente novamente. Por favor preencha um assunto Por favor preencha seu nome From c6ff80b334f40af7ae2f56b60a127b026734444d Mon Sep 17 00:00:00 2001 From: Matthias Wenz Date: Mon, 21 Nov 2016 21:09:19 +0100 Subject: [PATCH 12/16] Change Update button width to adapt to languages --- hockeysdk/src/main/res/layout/hockeyapp_activity_update.xml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hockeysdk/src/main/res/layout/hockeyapp_activity_update.xml b/hockeysdk/src/main/res/layout/hockeyapp_activity_update.xml index 6ab15e383..9e1fdd087 100644 --- a/hockeysdk/src/main/res/layout/hockeyapp_activity_update.xml +++ b/hockeysdk/src/main/res/layout/hockeyapp_activity_update.xml @@ -56,19 +56,20 @@