Merge branch 'development' of https://github.com/siacs/Conversations into development

kruks23 created

Change summary

AndroidManifest.xml                                            |  4 
README.md                                                      |  9 +
res/layout/dialog_verify_otr.xml                               |  3 
res/layout/edit_account_dialog.xml                             |  2 
res/values-de/strings.xml                                      |  2 
src/eu/siacs/conversations/services/XmppConnectionService.java | 10 ++
src/eu/siacs/conversations/ui/ContactsActivity.java            |  7 +
src/eu/siacs/conversations/ui/ConversationFragment.java        |  4 
src/eu/siacs/conversations/utils/UIHelper.java                 | 14 ---
src/eu/siacs/conversations/utils/Validator.java                |  2 
10 files changed, 25 insertions(+), 32 deletions(-)

Detailed changes

AndroidManifest.xml πŸ”—

@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="eu.siacs.conversations"
-    android:versionCode="12"
-    android:versionName="0.2.1" >
+    android:versionCode="13"
+    android:versionName="0.2.2" >
 
     <uses-sdk
         android:minSdkVersion="14"

README.md πŸ”—

@@ -42,11 +42,14 @@ These XEPs are - as of now:
 (In order of appearance)
 
 ###Code
-* Rene Treffer @rtreffer
-* Andreas Straub @strb
+* [Rene Treffer](https://github.com/rtreffer)
+* [Andreas Straub](https://github.com/strb)
 
 ###Translations
-* @beriain (Spanish and Basque)
+* [Sergio CΓ‘rdenas](https://github.com/kruks23) (Spanish)
+* [Benoit Bouvarel](https://github.com/BenoitBouvarel) (French)
+* [Daniel Gultsch](https://github.com/iNPUTmice) (German)
+
 
 ##FAQ
 ###General

res/layout/dialog_verify_otr.xml πŸ”—

@@ -20,7 +20,6 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:paddingLeft="8dp"
-        android:text="julia@jabber.example.com"
         android:textSize="14sp" />
      <TextView
         android:layout_width="wrap_content"
@@ -35,7 +34,6 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:paddingLeft="8dp"
-        android:text="2674D6A0 0B1421B1 BFC42AEC C56F3719 672437D8"
         android:textSize="14sp"
         android:typeface="monospace"/>
      <TextView
@@ -51,7 +49,6 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:paddingLeft="8dp"
-        android:text="2674D6A0 0B1421B1 BFC42AEC C56F3719 672437D8"
         android:textSize="14sp"
         android:typeface="monospace"/>
 </LinearLayout>

res/layout/edit_account_dialog.xml πŸ”—

@@ -19,7 +19,7 @@
         android:layout_height="wrap_content"
         android:ems="10"
         android:inputType="textEmailAddress"
-        android:hint="username@jabber.example.com">
+        android:hint="username@example.com">
 
 
     </EditText>

res/values-de/strings.xml πŸ”—

@@ -4,7 +4,7 @@
     <string name="app_name">Conversations</string>
     <string name="action_settings">Einstellungen</string>
     <string name="action_add">Neue Unterhaltung</string>
-    <string name="action_accounts">Kontos verwalten</string>
+    <string name="action_accounts">Konten verwalten</string>
     <string name="action_refresh">Kontaktliste neu laden</string>
     <string name="action_end_conversation">Unterhaltung beenden</string>
     <string name="action_contact_details">Kontaktdetails</string>

src/eu/siacs/conversations/services/XmppConnectionService.java πŸ”—

@@ -1156,8 +1156,11 @@ public class XmppConnectionService extends Service {
 		this.statusListener.onStatusChanged(account);
 		databaseBackend.updateAccount(account);
 		reconnectAccount(account, false);
-		if (accountChangedListener != null)
+		if (accountChangedListener != null) {
 			accountChangedListener.onAccountListChangedListener();
+		}
+		UIHelper.showErrorNotification(getApplicationContext(),
+				getAccounts());
 	}
 
 	public void deleteAccount(Account account) {
@@ -1166,8 +1169,11 @@ public class XmppConnectionService extends Service {
 		}
 		databaseBackend.deleteAccount(account);
 		this.accounts.remove(account);
-		if (accountChangedListener != null)
+		if (accountChangedListener != null) {
 			accountChangedListener.onAccountListChangedListener();
+		}
+		UIHelper.showErrorNotification(getApplicationContext(),
+				getAccounts());
 	}
 
 	public void setOnConversationListChangedListener(

src/eu/siacs/conversations/ui/ContactsActivity.java πŸ”—

@@ -515,9 +515,10 @@ public class ContactsActivity extends XmppActivity {
 			getActionBar().setHomeButtonEnabled(false);
 		}
 		this.rosterContacts.clear();
-		for (int i = 0; i < accounts.size(); ++i) {
-			rosterContacts.addAll(xmppConnectionService.getRoster(accounts
-					.get(i)));
+		for(Account account : accounts) {
+			if (account.getStatus() != Account.STATUS_DISABLED) {
+				rosterContacts.addAll(xmppConnectionService.getRoster(account));
+			}
 		}
 		updateAggregatedContacts();
 	}

src/eu/siacs/conversations/ui/ConversationFragment.java πŸ”—

@@ -583,8 +583,7 @@ public class ConversationFragment extends Fragment {
 		ConversationActivity activity = (ConversationActivity) getActivity();
 		if (this.conversation != null) {
 			for (Message message : this.conversation.getMessages()) {
-				if ((message.getEncryption() == Message.ENCRYPTION_PGP)
-						&& (message.getStatus() == Message.STATUS_RECIEVED)) {
+				if (message.getEncryption() == Message.ENCRYPTION_PGP) {
 					decryptMessage(message);
 					break;
 				}
@@ -758,7 +757,6 @@ public class ConversationFragment extends Fragment {
 
 	private class BitmapCache {
 		private HashMap<String, Bitmap> bitmaps = new HashMap<String, Bitmap>();
-		private Bitmap error = null;
 
 		public Bitmap get(String name, Contact contact, Context context) {
 			if (bitmaps.containsKey(name)) {

src/eu/siacs/conversations/utils/UIHelper.java πŸ”—

@@ -12,7 +12,6 @@ import eu.siacs.conversations.entities.Account;
 import eu.siacs.conversations.entities.Contact;
 import eu.siacs.conversations.entities.Conversation;
 import eu.siacs.conversations.entities.Message;
-import eu.siacs.conversations.entities.MucOptions;
 import eu.siacs.conversations.entities.MucOptions.User;
 import eu.siacs.conversations.ui.ConversationActivity;
 import eu.siacs.conversations.ui.ManageAccountActivity;
@@ -27,7 +26,6 @@ import android.content.DialogInterface;
 import android.content.DialogInterface.OnClickListener;
 import android.content.Intent;
 import android.content.SharedPreferences;
-import android.content.res.Resources;
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
 import android.graphics.Canvas;
@@ -38,11 +36,9 @@ import android.net.Uri;
 import android.preference.PreferenceManager;
 import android.provider.ContactsContract.Contacts;
 import android.support.v4.app.NotificationCompat;
-import android.support.v4.app.NotificationCompat.InboxStyle;
 import android.support.v4.app.TaskStackBuilder;
 import android.text.Html;
 import android.util.DisplayMetrics;
-import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.widget.LinearLayout;
@@ -234,11 +230,7 @@ public class UIHelper {
 		}
 		return bitmap;
 	}
-
-	private static Bitmap getUnknownContactPicture(String[] names, int size) {
-		return getUnknownContactPicture(names, size, UIHelper.BG_COLOR, UIHelper.FG_COLOR);
-	}
-
+	
 	private static Bitmap getMucContactPicture(Conversation conversation, int size, int bgColor, int fgColor) {
 		List<User> members = conversation.getMucOptions().getUsers();
 		if (members.size() == 0) {
@@ -273,10 +265,6 @@ public class UIHelper {
 	}
 
 	public static Bitmap getContactPicture(Contact contact, int dpSize, Context context, boolean notification) {
-		int fgColor = UIHelper.FG_COLOR,
-			bgColor = (notification) ?
-				UIHelper.BG_COLOR : UIHelper.TRANSPARENT;
-
 		String uri = contact.getProfilePhoto();
 		if (uri==null) {
 			return getContactPicture(contact.getDisplayName(), dpSize,

src/eu/siacs/conversations/utils/Validator.java πŸ”—

@@ -5,7 +5,7 @@ import java.util.regex.Pattern;
 
 public class Validator {
 	public static final Pattern VALID_JID = 
-		    Pattern.compile("^[A-Z0-9._%+-]+@[A-Z0-9.-]+\\.[A-Z]{2,6}$", Pattern.CASE_INSENSITIVE);
+		    Pattern.compile("\\b^[A-Z0-9._%+-]+@([A-Z0-9.-]+\\.)?\\d{1,3}[.]\\d{1,3}[.]\\d{1,3}[.]\\d{1,3}\\b$|^[A-Z0-9._%+-]+@[A-Z0-9.-]+\\.[A-Z]{2,6}$", Pattern.CASE_INSENSITIVE);
 	
 	public static boolean isValidJid(String jid) {
 		Matcher matcher = VALID_JID.matcher(jid);