- First commit to issue fork.
- @smustgrave opened merge request.
-
smustgrave β
committed e0daf777 on 8.x-1.x
Issue #3178388: Switching Slack.php properties from private to protected
-
smustgrave β
committed e0daf777 on 8.x-1.x
- πΊπΈUnited States smustgrave
Went ahead and made the properties protected to be inline with Drupal standards.
- Status changed to Downport
almost 2 years ago 10:53pm 24 January 2023 - πΊπΈUnited States tr Cascadia
This is also a problem in 2.0.x with SlackSendRequest.php
- Status changed to Postponed: needs info
over 1 year ago 6:56am 9 March 2023 In SlackSendRequest.php properties already protected.
There're only missing typehints, but it is another issue.
Can you provide some additional information?- Status changed to Downport
over 1 year ago 8:21pm 9 March 2023 - πΊπΈUnited States tr Cascadia
$ grep -r private src/Service/SlackSendRequest.php: private function processMessage(string $message) {
In the fact, function processMessage is internal helper function which not used anywhere outside class and it is a reason to be private.
sendMessage function can overriden and don't need to use internal helper function in it.If you don't think so then private modifier should be excluded from all OOP languages.
But it used in some other contrib modules.
For example in Config split module:\Drupal\config_split\ConfigSplitCliService::import
And I think function modifier shouldn't be changed.
- Status changed to Fixed
over 1 year ago 5:30am 14 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.