Skip to content
GitLab
Explorer
Connexion
S'inscrire
Navigation principale
Rechercher ou aller à…
Projet
S
stats-facil
Gestion
Activité
Membres
Labels
Programmation
Tickets
Tableaux des tickets
Jalons
Wiki
Code
Requêtes de fusion
Dépôt
Branches
Validations
Étiquettes
Graphe du dépôt
Comparer les révisions
Extraits de code
Compilation
Pipelines
Jobs
Planifications de pipeline
Artéfacts
Déploiement
Releases
Registre de paquets
Registre de conteneur
Registre de modèles
Opération
Environnements
Modules Terraform
Surveillance
Incidents
Analyse
Données d'analyse des chaînes de valeur
Analyse des contributeurs
Données d'analyse CI/CD
Données d'analyse du dépôt
Expériences du modèle
Aide
Aide
Support
Documentation de GitLab
Comparer les forfaits GitLab
Forum de la communauté
Contribuer à GitLab
Donner votre avis
Raccourcis clavier
?
Extraits de code
Groupes
Projets
Afficher davantage de fils d'Ariane
facil
stats-facil
Validations
6775356f
Valider
6775356f
rédigé
10 years ago
par
mattab
Parcourir les fichiers
Options
Téléchargements
Correctifs
Plain Diff
New method getAllUsersPreferences and test
parent
6156633b
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
Modifications
2
Masquer les modifications d'espaces
En ligne
Côte à côte
Affichage de
2 fichiers modifiés
plugins/UsersManager/API.php
+28
-1
28 ajouts, 1 suppression
plugins/UsersManager/API.php
plugins/UsersManager/tests/Integration/APITest.php
+39
-0
39 ajouts, 0 suppression
plugins/UsersManager/tests/Integration/APITest.php
avec
67 ajouts
et
1 suppression
plugins/UsersManager/API.php
+
28
−
1
Voir le fichier @
6775356f
...
...
@@ -32,6 +32,8 @@ use Piwik\Tracker\Cache;
*/
class
API
extends
\Piwik\Plugin\API
{
const
OPTION_NAME_PREFERENCE_SEPARATOR
=
'_'
;
/**
* @var Model
*/
...
...
@@ -107,9 +109,34 @@ class API extends \Piwik\Plugin\API
return
$this
->
getDefaultUserPreference
(
$preferenceName
,
$userLogin
);
}
/**
* Returns an array of Preferences
* @param $preferenceNames array of preference names
* @return array
* @ignore
*/
public
function
getAllUsersPreferences
(
array
$preferenceNames
)
{
Piwik
::
checkUserHasSuperUserAccess
();
$userPreferences
=
array
();
foreach
(
$preferenceNames
as
$preferenceName
)
{
$optionNameMatchAllUsers
=
$this
->
getPreferenceId
(
'%'
,
$preferenceName
);
$preferences
=
Option
::
getLike
(
$optionNameMatchAllUsers
);
foreach
(
$preferences
as
$optionName
=>
$optionValue
)
{
$optionName
=
explode
(
self
::
OPTION_NAME_PREFERENCE_SEPARATOR
,
$optionName
);
$userName
=
$optionName
[
0
];
$preference
=
$optionName
[
1
];
$userPreferences
[
$userName
][
$preference
]
=
$optionValue
;
}
}
return
$userPreferences
;
}
private
function
getPreferenceId
(
$login
,
$preference
)
{
return
$login
.
'_'
.
$preference
;
return
$login
.
self
::
OPTION_NAME_PREFERENCE_SEPARATOR
.
$preference
;
}
private
function
getDefaultUserPreference
(
$preferenceName
,
$login
)
...
...
Ce diff est replié.
Cliquez pour l'agrandir.
plugins/UsersManager/tests/Integration/APITest.php
+
39
−
0
Voir le fichier @
6775356f
...
...
@@ -69,4 +69,43 @@ class APITest extends IntegrationTestCase
$this
->
assertFalse
(
$eventTriggered
,
'UsersManager.removeSiteAccess event was triggered but should not'
);
}
public
function
test_getAllUsersPreferences_isEmpty_whenNoPreference
()
{
$preferences
=
$this
->
api
->
getAllUsersPreferences
(
array
(
'preferenceName'
));
$this
->
assertEmpty
(
$preferences
);
}
public
function
test_getAllUsersPreferences_isEmpty_whenNoPreferenceAndMultipleRequested
()
{
$preferences
=
$this
->
api
->
getAllUsersPreferences
(
array
(
'preferenceName'
,
'otherOne'
));
$this
->
assertEmpty
(
$preferences
);
}
public
function
test_getAllUsersPreferences_shouldGetMultiplePreferences
()
{
$user2
=
'userLogin2'
;
$user3
=
'userLogin3'
;
$this
->
api
->
addUser
(
$user2
,
'password'
,
'userlogin2@password.de'
);
$this
->
api
->
setUserPreference
(
$user2
,
'myPreferenceName'
,
'valueForUser2'
);
$this
->
api
->
setUserPreference
(
$user2
,
'Random_NOT_REQUESTED'
,
'Random_NOT_REQUESTED'
);
$this
->
api
->
addUser
(
$user3
,
'password'
,
'userlogin3@password.de'
);
$this
->
api
->
setUserPreference
(
$user3
,
'myPreferenceName'
,
'valueForUser3'
);
$this
->
api
->
setUserPreference
(
$user3
,
'otherPreferenceHere'
,
'otherPreferenceVALUE'
);
$this
->
api
->
setUserPreference
(
$user3
,
'Random_NOT_REQUESTED'
,
'Random_NOT_REQUESTED'
);
$expected
=
array
(
$user2
=>
array
(
'myPreferenceName'
=>
'valueForUser2'
),
$user3
=>
array
(
'myPreferenceName'
=>
'valueForUser3'
,
'otherPreferenceHere'
=>
'otherPreferenceVALUE'
,
),
);
$result
=
$this
->
api
->
getAllUsersPreferences
(
array
(
'myPreferenceName'
,
'otherPreferenceHere'
,
'randomDoesNotExist'
));
$this
->
assertSame
(
$expected
,
$result
);
}
}
Ce diff est replié.
Cliquez pour l'agrandir.
Aperçu
0%
Chargement en cours
Veuillez réessayer
ou
joindre un nouveau fichier
.
Annuler
You are about to add
0
people
to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Enregistrer le commentaire
Annuler
Veuillez vous
inscrire
ou vous
se connecter
pour commenter