fix: refer to portnumber property correctly (#209)

* fix: refer to portnumber property correctly * chore: add changelogs
lowlydba · Oct 7, 2023 · f78faaf · f78faaf
1 parent 787f403
commit f78faaf
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 2 deletions.
diff --git a/changelogs/fragments/209-fix-port-ref.yml b/changelogs/fragments/209-fix-port-ref.yml
@@ -0,0 +1,2 @@
+bugfixes:
+  - Get port properly from AG Listener when comparing to input port value.
diff --git a/changelogs/fragments/release-summary-2-2-2.yml b/changelogs/fragments/release-summary-2-2-2.yml
@@ -0,0 +1 @@
+release_summary: Minor bugfix for setting the port on an AG listener.
diff --git a/galaxy.yml b/galaxy.yml
@@ -2,7 +2,7 @@
 
 namespace: lowlydba
 name: sqlserver
-version: 2.2.1
+version: 2.2.2
 readme: README.md
 authors:
   - John McCall (github.com/lowlydba)

diff --git a/plugins/modules/ag_listener.ps1 b/plugins/modules/ag_listener.ps1
@@ -63,7 +63,7 @@ try {
             $output = Add-DbaAgListener @listenerParams
             $module.Result.changed = $true
         }
-        elseif ($existingListener.Port -ne $port) {
+        elseif ($existingListener.PortNumber -ne $port) {
             $output = Set-DbaAgListener -AvailabilityGroup $agName -Listener $listenerName -Port $port
             $module.Result.changed = $true
         }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		bugfixes:
		- Get port properly from AG Listener when comparing to input port value.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		release_summary: Minor bugfix for setting the port on an AG listener.