Change-Id: I796fa45040bc7f04ef7ecb34265dda874f59a80c

Change-Id: I174ebd3019a87cc1992b5ea91c058c2a8758a73a
Reviewed-by: Anthony Léonard <anthony.leonard@savoirfairelinux.com>

add the contact addition wrapping from ConfigurationManager.
add hasContact to check if a contact method is part account contacts.

Change-Id: I10cd59ebdba74dba5098d794810f29542b70daa4

Sébastien Blin authored 7 years ago and Philippe Gorley committed 7 years ago

When a call failed, the call should be refused automatically after some time

Change-Id: I9b4772d55a3b185f8fe5980535b6f4cdf0879a49
Reviewed-by: Philippe Gorley <philippe.gorley@savoirfairelinux.com>

Sébastien Blin authored 7 years ago and Philippe Gorley committed 7 years ago

PeerProfileCollection's name is "Peer profiles" not "ppc"

Change-Id: I3ad0f400c01b404161f87df6f7288baadd4c4ce8
Reviewed-by: Philippe Gorley <philippe.gorley@savoirfairelinux.com>

Change-Id: I182d2cf3349fe444765727933a64aa47a9c04a7b
Reviewed-by: Anthony Léonard <anthony.leonard@savoirfairelinux.com>

Create a removeContact method to allow an account to remove contacts.

Change-Id: I7145f0ead20cd194a70e1978d1cb547a4898abb0

Change-Id: Ie560fab55336079277a710380e25d0889889606a

Change-Id: I08da4e9afe94f94a37efb6edab2a7a4024471763

Sébastien Blin authored 7 years ago and Guillaume Roguez committed 7 years ago

Now, resolutions are sorted by area, not by alphabetical order.

Change-Id: Iaa9ef15c6009c3884628d2177493ef73ad98a037
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Change-Id: Ic67a7ab2aef99c08f16112314a750d044946dd3d

Change-Id: I90117abd50199e63c78e318115cafd3317b576b9

Guillaume Roguez authored 7 years ago and Adrien Béraud committed 7 years ago

pointers used without checking for nullptr value.

Change-Id: I63eb9b460b44264857ced9ae0aff9bbd634b241c

Guillaume Roguez authored 7 years ago and Adrien Béraud committed 7 years ago

Daemon ContactAdded signal is used to add/update contact information
like if it "trusted" by user or by peer (i.e. 'confirmed' property).
Create new Contact method using this signal and add it
to account daemon contacts list.

Change-Id: I76a3fdd9b92380c16e205ca437e25d977f0b1c11

Guillaume Roguez authored 7 years ago and Adrien Béraud committed 7 years ago

Add isConfirmed/setConfirmed property saying if a contact method
is confirmed as trusted by the associated peer account.

Synchronized them on daemon contact property 'confirmed' during
the account loading.

Change-Id: I04bb1b56afea43b2fe02b242c43d9d75fe4e1e44

Guillaume Roguez authored 7 years ago and Kateryna Kostiuk committed 7 years ago

The application crashes if the ContactMethod given to
BannedContactModel::remove() doesn't exist in the banned list.
Check for existance and no-op if not.

Change-Id: Ifde1c1b9a0c369889c8dabfce20dd8d8937b715b
Reviewed-by: Kateryna Kostiuk <kateryna.kostiuk@savoirfairelinux.com>

Sometimes contact method could be added to BannedContactModel twice:
during creating model and from accountModel when receiving
slotContactRemoved.(it happens if account does not have bannedContactModel
yet). This patch fix it by checking if ContactMethod already exists in the
banned contact list.

Change-Id: I514e0301bbc07378ef9f9a5c345db5b49c8e23e0
Reviewed-by: Anthony Léonard <anthony.leonard@savoirfairelinux.com>

Change-Id: I4d2659e6798b9ea461d1e214b89da20e0c26f878

Change-Id: I2ab88a8269593925b8809de4aca8fd04a7c1b85f

Change-Id: I179baa7d15b66569b1adc63cfa85d83da5c79f83

Replace ContactMethod::BLANK() by ContactMethod::ContactMethod(),
the default constructor.

This is the general way to construct a instance with "defaults"
behaviours, like BLANK tries to do.

Also removes the nasty usage of BLANK:
- it returns a constant instance
- the code uses it with a const_cast<>, removing the only protection!
No sense and extremly dangerous to operate with such global and
non protected instance.

Side effect: Call::peerContactMethod() returns a nullptr
and not a valid and !!mutable!! global const instance if no cm exists.

Change-Id: I8a29eacad657b76a7670055f27afeb6db98d78dc
Reviewed-by: Olivier Soldano <olivier.soldano@savoirfairelinux.com>

my previous patch [cm: replace BLANK() by default ctor]
causes a nullptr access in ProfileModelPrivate::slotAccountAdded()
There is a call to acc->contactMethod() that returns a nullptr.

I really tried to understand this call and its impact...
... no success!
Removing the line seems to have no effect during tests on Gnome client.

LRC is magic!

Change-Id: Icdd33989e5d90bfb93227d82354643269fe944ff
Reviewed-by: Olivier Soldano <olivier.soldano@savoirfairelinux.com>

Emmanuel Lepage Vallée authored 7 years ago and Guillaume Roguez committed 7 years ago

Fixed in a ring-daemon commit a couple of days ago. Time to fix
it in LRC.

Why: I created a CI Docker image to compile ring-lrc using only
a curated list of dependency to avoid the linking bugs to come
back.

Change-Id: Ic76124b98fc3755f6b12be8b45b99932dff50c7c
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Nicolas Jager authored 7 years ago and Guillaume Roguez committed 7 years ago

[Guillaume: fix a nullptr access on account]
[Guillaume: less code line by using QVariant static function]
[Guillaume: few typo fixes]

Change-Id: I3f145cd0fdbcd02cd0820b27122abeaf52409164
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

- add a new class BannedContactModel. This model loads at start of the
client a list from the daemon to figure which ContactMethods are
banned.

- it updates the list (for now, we can only ban) when a peer is banned.

[Guillaume: fix typo in ci msg]

Change-Id: I2367c8918dd4f743ebbce2b0c1c90e5fde71c45b

[JN : add missing paramter]

Change-Id: Id76c75511e5601d5147858fc84caa39201d8f538
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>

Keep Call entries in PeopleProxy row filter.
This fixes the Conference entry display.

Change-Id: If2c4feb37536c37209add3e7c410cdc853f80475
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>

Nicolas Jager authored 7 years ago and Guillaume Roguez committed 7 years ago

Fix a regression introduced by:
[59f41f8d44: avoid adding entry in RecentModel after an incoming ContactRequest]

This patch check if the pointer is not null before using it.

Change-Id: I49b63f4044bbacc06b36e2c10a3fe7af29100477
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

- when we get an incoming ContactRequest we build a Person from the
vcard nested in the payload. We next set this Person object to a
ContactMethod which trigger the signal contactChanged. When this
signal is emitted, RecentModel adds an entry.

- this patch adds a check before adding an entry in the RecentModel,
if the current changed is for an actual contact or a ContactRequest.
In last case, it doesn't create a new entry.

Change-Id: I82c03ade965e65f5e32cdc7ecf81b527c4c3aeee
Reviewed-by: Anthony Léonard <anthony.leonard@savoirfairelinux.com>

Emmanuel Lepage Vallée authored 7 years ago and Guillaume Roguez committed 7 years ago

Change-Id: I584e311c2ffe9d3f705b0635ea88b5fbda25eb58
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Emmanuel Lepage Vallée authored 7 years ago and Guillaume Roguez committed 7 years ago

Change-Id: Iea3fa1d638698c74d425f7e5c0b31af7c1ce713c
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Olivier SOLDANO authored 7 years ago and Guillaume Roguez committed 7 years ago

Without this library it was impossible to acquire the framerate
model of any video camera on Windows platform (.dll)

[Guillaume: ci msg edited for details]

Change-Id: Ib06a680bc2a5c90210949a868dc7b0cb8adc64ba
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Emmanuel Lepage Vallée authored 7 years ago and Guillaume Roguez committed 7 years ago

Was found while reading the code

Change-Id: I4191ffa3c376156a9fbd686b4fad82fe8ead8f19
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Emmanuel Lepage Vallée authored 7 years ago and Guillaume Roguez committed 7 years ago

Change-Id: I0ce0b1666c8388880d56abc8f85865828721698a
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Stepan Salenikovich authored 7 years ago and Guillaume Roguez committed 7 years ago

In the case we replace a ContactMethod node by a Person node, we
need to make sure that the selection is updated. This is especially
important during an incoming Call.

Change-Id: I9bf78b7d359b7d01cc55b50178a13d703a72d6ba
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>

Missing a slot definition in commit
[5a4d864e: PhoneDirectoryModel: remove incomingMessage slot]

Change-Id: Ic03a5af3f4f289b69c8bbe0d7d25a1cded940f9c
Reviewed-by: Olivier Soldano <olivier.soldano@savoirfairelinux.com>

IM has nothing related to a PhoneDirectory.
IM signals are already handled by media::Text class.
Code duplication that may results to bugs.

Change-Id: Ie4d26d76d7267ab73a46f4fc5224a4b52e0466bd
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>

VCardUtils::parseMimeAttributes() returns by value, but
it was used in IMConversationManagerPrivate::newMessage() by reference.
So this reference is on a stack-allocated object and can be invalidated
by the system.

Use a by-value copy, mostly optimized by compiler with copy elision [1].

[1] http://en.cppreference.com/w/cpp/language/copy_elision



Change-Id: I1b7ef8eb1ebc738949243d5ace1363d4b4fa3caf
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>

Nicolas Jager authored 7 years ago and Kateryna Kostiuk committed 7 years ago

- add contactAdded and contactRemoved signal required by clients
using libwrap.

Change-Id: I809139cd103380605de86d61a4841d436393f67d
Reviewed-by: Kateryna Kostiuk <kateryna.kostiuk@savoirfairelinux.com>

Nicolas Jager authored 7 years ago and Kateryna Kostiuk committed 7 years ago

- getContactDetails is missing in configurationmanager_wrap.h leadig
clients without dbus to failed at compilation, when this function is
called.

Change-Id: I324fbd2644e9a5c72bb6468b7c63923c7e53535a
Reviewed-by: Kateryna Kostiuk <kateryna.kostiuk@savoirfairelinux.com>