{"action":"closed","issue":{"url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/issues/10","labels_url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/issues/10/labels{/name}","comments_url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/issues/10/comments","events_url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/issues/10/events","html_url":"https://github.com/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/issues/10","id":116409587,"number":10,"title":"New-MessageReceiver: Cleanup required","user":{"login":"dfch","id":8310360,"avatar_url":"https://avatars.githubusercontent.com/u/8310360?v=3","gravatar_id":"","url":"https://api.github.com/users/dfch","html_url":"https://github.com/dfch","followers_url":"https://api.github.com/users/dfch/followers","following_url":"https://api.github.com/users/dfch/following{/other_user}","gists_url":"https://api.github.com/users/dfch/gists{/gist_id}","starred_url":"https://api.github.com/users/dfch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dfch/subscriptions","organizations_url":"https://api.github.com/users/dfch/orgs","repos_url":"https://api.github.com/users/dfch/repos","events_url":"https://api.github.com/users/dfch/events{/privacy}","received_events_url":"https://api.github.com/users/dfch/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/dfensgmbh/biz.dfch.PS.Azure.ServiceBus.Client/labels/task","name":"task","color":"0052cc"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-11T19:56:41Z","updated_at":"2015-12-14T10:00:58Z","closed_at":"2015-12-14T10:00:58Z","body":"The code in `New-MessageReceiver.ps1` contains a `Credential` input parameter that is not really used.  This must be fixed.\r\n\r\nIn addition, there are code parts which are not maintainable (3line if statements) - fix as well.\r\n\r\nWhy is `PeekLock` the default behaviour? I think we agreed on `ReceiveAndDelete` as the default.\r\n\r\nIn general I would recommend to rename the Cmdlet from `New-MessageReceiver` to `Get-MessageReceiver` or something with `Factory` is this is actually more hinting at its use.\r\n\r\nInline help not consistent or giving meaningful information.\r\n\r\nWhy is this Cmdlet exported? does anyone external call this? or is it more like an internal Cmdlet?"}}