Skip to content

Commit

Permalink
UI-1262: We now display the presence_id as the main extension if conf…
Browse files Browse the repository at this point in the history
…igured
  • Loading branch information
JRMaitre committed Mar 10, 2015
1 parent bad49c1 commit c6de12f
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions submodules/users/users.js
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ define(function(require){
additionalExtensions: 0,
additionalNumbers: 0,
devices: [],
extension: '',
extension: dataUser.hasOwnProperty('presence_id') ? dataUser.presence_id : '',
hasFeatures: false,
isAdmin: dataUser.priv_level === 'admin',
listCallerId: [],
Expand Down Expand Up @@ -253,13 +253,6 @@ define(function(require){
_.each(callflow.numbers, function(number) {
if(number.length < 7) {
user.extra.listExtensions.push(number);

if(user.extra.extension === '') {
user.extra.extension = number;
}
else {
user.extra.additionalExtensions++;
}
}
else {
user.extra.listCallerId.push(number);
Expand All @@ -274,6 +267,15 @@ define(function(require){
}
}
});

// The additional extensions show how many more extensions than 1 a user has.
// So if the user has at least 1 extension, then we count how many he has minus the one we already display, otherwise we display 0.
user.extra.additionalExtensions = user.extra.listExtensions.length >= 1 ? user.extra.listExtensions.length - 1 : 0;

// If the main extension hasn't been defined because the presence_id isn't set, just pick the first extension
if(user.extra.extension === '' && user.extra.listExtensions.length > 0) {
user.extra.extension = user.extra.listExtensions[0];
}
}
}
});
Expand Down Expand Up @@ -1403,8 +1405,6 @@ define(function(require){
}
});

console.log(orderedArray);

_.each(orderedArray, function(number) {
var currentNumber = parseInt(number);

Expand Down

0 comments on commit c6de12f

Please sign in to comment.