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>

Due to the ability to receive vCards from other peers (during a
Call or from a ContactRequest), there are many possibilities to
create duplicate Person objects. This is because up until now we
have been using the Person's UID as a unique identifier. However
the UID has come from the vCard, and we cannot trust the vCards we
receive from peers to have either unique UIDs when they are
different contacts, nor the same UID if its the same contact.

Thus in the case of received vCards, we identify whether or not
a new Person should be created based on the ContactMethod.

Note that this doesn't necessarily solve the issue in the case
we have multiple ContactMethods with the same RingID (this may
happen when we have multiple Accounts communicating with the same
peer). However, this is a step in the right direction.

Change-Id: I9c05fe00fc46ba0e26e8886cad33cf0fd4afeb81

This fixes one particular instance of "contact duplication" in the
RecentModel. When we had a call from a peer incoming for the very
first time, a ContactMethod ndoe was first being created. Then we
received the profile from the peer, and so a Person node was created.
However, the contactChanged signal could be called before the Person
node is created; in this case the ContactMethod node was not being
removed and the result was a duplication of the the same contact
twice in the RecentModel: once as a CM and once as a Person.

Calling slotLastUsedTimeChanged() on the new Person ensures that
the Person node is created.

We also call slotContactChanged() from slotPersonAdded() to reduce
code duplication.

Change-Id: Ia9b49ab939a017052cccc69f8d72df5db0aef41d

Use the DRing::Account::TrustRequest constants instead

Change-Id: I0f6e1e71c21fe689f071abe6d3ae5afed9a71926
Reviewed-by: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>

It returns the bestId() of the lastUsedContactMethod().
This way views will not get an empty QVariant when asking for the
number role from a Person item.

This also makes sense as what we want to show most of the time and
most of the time a Person will only have one ContactMethod anyways.

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

Changes the function to return the pointer to the last used
ContactMethod. This is more generic than returning the id, since
the caller can then decide what info they need themselves.

Drops the "get" from the function name to follow LRC/Qt
convention: getters don't begin with "get".

Makes this getter public as its useful for other purposes as well.

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

Instead of sending the vCard during an incoming or ringing call,
we wait to send the vCard only once the call has started. This
is to prevent connectivity failures during the call initialization
due to dropped vCard packets.

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

This moves the logic for determining the best name to use away
from the roleData function into its own function. This way we
reduce the logic in roleDat() and make it easy to access this
property if we have the pointer to a ContactMethod.

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

This is to follow LRC/Qt naming convention. Getters don't begin
with the word "get".

Note that getBestId() has been kept so as not to break API, but
marked as deprecated so that we can quickly update the clients.

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

Calls and ContactMethods will support this role in case the user
would like to explicitly get the URI and not the registered name.

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