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 : Default action type code when send mail. #29527

Closed

Conversation

atm-irvine
Copy link
Contributor

FIX|Fix Default action type code when send mail. Changed from AC_OTH_AUTO to AC_EMAIL

By default, the action type code when you send an email is AC_OTH_AUTO. I think it is more relevant to set it to AC_EMAIL.

@@ -382,7 +382,7 @@
// Initialisation of datas of object to call trigger
if (is_object($object)) {
if (empty($actiontypecode)) {
$actiontypecode = 'AC_OTH_AUTO'; // Event insert into agenda automatically
$actiontypecode = 'AC_EMAIL'; // Event insert into agenda automatically
Copy link
Member

@eldy eldy Apr 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is 2 types of events:
Event enter manually by the user into the agenda.
Events created automatically by the system for tracking purposes.

An email send from a screen is the second one. So it must not appears into the agenda of the user among its rendez vous but into the agenda of tracking events. So $actiontypecode (the type auto / non auto) must be 'AC_OTH_AUTO' (instead of AC_OTH). All other codes inside llx_c_actioncomm are deprecated.

Note that there is a column "code" into the table llx_actioncomm that is what king of action has been recorded. You can know it is an email with this code, because emails are codes like 'AC_..._SENTBYMAIL' into this table.

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Apr 29, 2024
@atm-irvine atm-irvine closed this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants