Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX compatibility with payment type sharing (Multicompany) #32800

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

hregis
Copy link
Contributor

@hregis hregis commented Jan 26, 2025

No description provided.

@hregis hregis requested review from eldy and atm-florianm January 26, 2025 10:30
@hregis
Copy link
Contributor Author

hregis commented Jan 26, 2025

@mdeweerd how to fix this phan error ? thanks

@mdeweerd
Copy link
Contributor

I already fied several notices in #32779 - I do not know which phan/phpstan notice you are interested in.

The only phan notice remaining in that PR is:

htdocs/comm/action/peruser.php#L1352
peruser.php: PhanTypeMismatchProperty: Assigning ($userId as a field) of type array<int,non-zero-int> to property but \ActionComm->userassigned is array<int,array{id:int,transparency:int}>

where ActionComm->userassigned is an array of arrays but a simple array is assigned.
That type for userassigned could be incorrect, but if it is incorrect then I think that other code using 'userassigned' also needs updates.

@hregis
Copy link
Contributor Author

hregis commented Jan 27, 2025

@eldy if it's ok, can I move this code into a new function in "functions.lib.php" in order to use it in other dictionaries, with a hook inside to have the possibility to customize it?
or in multicompany module !

@hregis hregis force-pushed the fix_dev_select_payment_type_mc branch from 62582d7 to e7cf057 Compare January 27, 2025 08:15
@hregis
Copy link
Contributor Author

hregis commented Jan 27, 2025

je ne vais pas passer mon temps à corriger les erreurs phpstan des autres fichiers qui ne concernent pas ma PR !! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants