Le Propriété de get-adgroupmember est Un singleton, pas un tableau . Toutefois, la propriété accepte le pipeline d'entrée. Donc, vous pourrez peut-être faire quelque chose comme ceci: xxx Bien que, IMX, certaines des commandes de l'annonce sont un peu bonsky en raison de leur âge. J'attends que vous ayez peut-être besoin de faire quelque chose comme ceci: xxx Le reste de votre script a une sorte de logique confuse, cependant, je ne peux donc pas vraiment dire ce que vous voulez vraiment dire ce que vous " retenir de faire.
{snip}
basé sur vos commentaires, voici ce que je ferais.
Tout d'abord, je changerais de fichier de groupe. Au lieu d'une liste de texte brut des groupes, j'aurais un fichier CSV avec deux colonnes: l'ancien groupe et le nouveau groupe.
donc, grouplist.csv looks comme ceci: xxx Vous avez maintenant une carte pour chaque groupe ancien et le groupe que vous souhaitez migrer vos utilisateurs.
maintenant, nous le faisons Comme ceci: xxx [Remarque: retirez le -Qu- ut pour effectuer réellement les actions.] Pour chaque groupe, nous obtenons une liste des membres des groupes, filtrez-le aux noms d'utilisateur dans $ utilisateur et enregistrez-le sur $ USERTOMOFY . Ensuite, nous transmettons cette liste des utilisateurs vers les commandes Supprimer et ajouter. Nous n'avons besoin que de l'appeler une fois par groupe. Je sais que vous avez eu une exception spéciale pour Win7 à Win10, mais je ne vois pas où la logique du script doit vraiment changer pour accueillir cela. Si vous souhaitez toujours ajouter tous les utilisateurs dans $ utilisateurs à Win10, vous pouvez l'ajouter manuellement: xxx
À quoi ressemble les membres $? Il semble que des membres $ devraient être quelque chose comme ... Membres $ = foreach ($ g en groupes de dollars) {get-adgroupmember -sidité $ Groupe.someProperty}
Je ne suis pas sûr de ce que tu veux dire? $ Membres n'est qu'une variable telle que définie dans le même sommet du script.
Je pense que
m0lochwalker demande quelle est la structure du contenu de ce $ var. Vous appelez le code pour celui-ci avec une collection de groupes ... DOCS Dites[- identité] ... et cette valeur de paramètre est un groupe, pas une collection de groupes . Lee, je n'utilise pas correctement grouplist.txt pour la collection de groupes?
@Jeff - Pour obtenir votre MSG à une personne en particulier, utilisez
@ et leur nom tel qu'il apparaît lorsque vous le faites. [ grin ] ///// oui, vous utilisez la cmdlet incorrectement commeM0Lochwalker points de sortie ... Vous utilisez une collection où le Docs indiquent clairement que vous devez avoir un élément. L'exemple de code posté parm0lochwalker semble fonctionner de fonctionner lorsqu'il est ajusté pour adapter votre fichier source de données.@Jeff donc, $ Membres n'est pas simplement une variable ... c'est la sortie de votre cmdlet à laquelle vous vous assignez à $ membres. Plus tard dans votre message, vous utilisez $ MEMBRES.SAMAccountName. Membres $ $ [0] vous donne-t-il une collection de propriétés pour cet objet, l'un d'entre eux étant SamAccountName?
Quelque chose comme $ membres = $ groupe | foreach-objet {get-adgroupmembermembermembermemberd $ _} pourrait aussi vous convenir.
Vous utilisez également la même chose si l'état encore et plus. Peut aussi bien faire tout enlever / ajouter quoi que ce soit, puisque votre condition ne change jamais. On dirait que vous devriez repenser beaucoup de script. Je peux aider quand je ne suis pas en mobile.
@ M0Lochwalker, si vous pouviez m'aider à proposer une logique qui fonctionnerait pour mon objectif, je l'apprécierais vraiment!
@ M0LOCHWALKER Je veux dire que je sais que je sais qu'un moyen serait d'avoir une instruction si une instruction et d'exécuter le script à plusieurs reprises, en modifiant le nom du groupe avec le nom du groupe durement codé, mais c'est ce que j'essaie d'empêcher de devoir faire.
@Jeff j'ai fourni une réponse ci-dessous. Soyez très sûr que vous testez d'abord sur un petit groupe de contrôle. Merci!