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

NEW contact type on auto add contributor #26077

Merged
merged 10 commits into from
Feb 21, 2024

Conversation

BenjaminFlr
Copy link
Contributor

For now, if an user creates a new ticket and is automatically added as a contributor, it's as an internal contributor even if it's an external user.

I propose to make the difference between internal and external for automatic add.

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Sep 30, 2023
@BenjaminFlr
Copy link
Contributor Author

@eldy : I made the change but it looks like Travis is unhappy for some code not related to my PR.

@BenjaminFlr BenjaminFlr requested a review from eldy October 2, 2023 12:57
@hregis
Copy link
Contributor

hregis commented Oct 2, 2023

@altairis-benjamin a "var_dump" line 87 in "/htdocs/core/ajax/bankconciliate.php"

@BenjaminFlr
Copy link
Contributor Author

@altairis-benjamin a "var_dump" line 87 in "/htdocs/core/ajax/bankconciliate.php"

Oui mais il ne vient pas de chez moi, pour le coup ;) C'est pour ça que je ne l'ai pas touché. Je le supprime et je push à nouveau @hregis ?

@hregis
Copy link
Contributor

hregis commented Oct 2, 2023

@altairis-benjamin fait un pull, Laurent a corrigé l'erreur

@BenjaminFlr
Copy link
Contributor Author

I think I'll make new changes as the activation of TICKET_ADD_AUTHOR_AS_CONTACT prevent public interface from working with inexistant users like a simple contact without any access to the dolibarr.

What do you think about that @eldy ?

@eldy eldy added PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) and removed Discussion Some questions or discussions are opened and wait answers of author or other people to be processed labels Oct 4, 2023
@BenjaminFlr
Copy link
Contributor Author

Hello @eldy,
I just fixed my PR with the use of GetDolGlobalInt.
I added a context for tickets created from the public interface and defined it as a requirement to the constant TICKET_ADD_AUTHOR_AS_CONTACT. Indeed, without that check, it will block the ticket creation from public interface.

I also added the ticket module by default to the MAIN_MODULES_FOR_EXTERNAL constant, because I assume that it's quite standard.

@BenjaminFlr
Copy link
Contributor Author

Little up here @eldy ? :)

@eldy eldy merged commit 5460b97 into Dolibarr:develop Feb 21, 2024
4 of 5 checks passed
@BenjaminFlr BenjaminFlr deleted the contributorType branch February 29, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants