Skip to content

Issue #239557 Fix : IT is SHOWING error when we try to save I have just added attendee name tag so that in mail attendee name should display. #178

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: release-3.1.0
Choose a base branch
from
Open

Issue #239557 Fix : IT is SHOWING error when we try to save I have just added attendee name tag so that in mail attendee name should display. #178

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions src/com_tjnotifications/admin/language/en-GB.com_tjnotifications.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -311,3 +311,17 @@ COM_TJNOTIFICATIONS_SETTINGS_WEBHOOK_URL_LABEL="Webhook URL"
COM_TJNOTIFICATIONS_SETTINGS_WEBHOOK_URL_DESC="Enter URL for your Webhook notification"
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_CUSTOM_WEBHOOK_URLS="Please add custom webhook URL or use global webhook URL"
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_GLOBAL_WEBHOOK_URLS="Please add webhook URLs in the global config or add custom webhook URL."

;Since 3.1.0
;Notification - Validation error messages
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_WEBHOOK_BODY_REQUIRED="Webhook body is required when webhook is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_WEBHOOK_LANGUAGE_REQUIRED="Language selection is required when webhook is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_EMAIL_BODY_REQUIRED="Email body is required when email is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_EMAIL_SUBJECT_REQUIRED="Email subject is required when email is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_EMAIL_LANGUAGE_REQUIRED="Language selection is required when email is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_SMS_BODY_REQUIRED="SMS body is required when SMS is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_SMS_LANGUAGE_REQUIRED="Language selection is required when SMS is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_PUSH_BODY_REQUIRED="Push body is required when push is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_PUSH_LANGUAGE_REQUIRED="Language selection is required when push is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_WHATSAPP_BODY_REQUIRED="WhatsApp body is required when WhatsApp is enabled."
COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_WHATSAPP_LANGUAGE_REQUIRED="Language selection is required when WhatsApp is enabled."
3 changes: 0 additions & 3 deletions src/com_tjnotifications/admin/models/forms/emailfields.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
type="contentlanguage"
label="JFIELD_LANGUAGE_LABEL"
description="COM_TJNOTIFICATIONS_NOTIFICATION_LANGUAGE_DESC"
required="true"
>
<option value="*">JALL</option>
</field>
Expand All @@ -20,7 +19,6 @@
type="textarea"
label="COM_TJNOTIFICATIONS_FIELD_EMAIL_SUBJECT_LABEL"
description="COM_TJNOTIFICATIONS_FIELD_EMAIL_SUBJECT_DESC"
required="true"
/>

<field
Expand All @@ -31,7 +29,6 @@
buttons="true"
filter="JComponentHelper::filterText"
height="150px"
required="true"
/>
</fieldset>
</form>
2 changes: 0 additions & 2 deletions src/com_tjnotifications/admin/models/forms/pushfields.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
type="contentlanguage"
label="JFIELD_LANGUAGE_LABEL"
description="COM_TJNOTIFICATIONS_NOTIFICATION_LANGUAGE_DESC"
required="true"
>
<option value="*">JALL</option>
</field>
Expand All @@ -24,7 +23,6 @@
cols="150"
filter="JComponentHelper::filterText"
class="validate-json"
required="true"
/>
</fieldset>
</form>
2 changes: 0 additions & 2 deletions src/com_tjnotifications/admin/models/forms/smsfields.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
type="contentlanguage"
label="JFIELD_LANGUAGE_LABEL"
description="COM_TJNOTIFICATIONS_NOTIFICATION_LANGUAGE_DESC"
required="true"
>
<option value="*">JALL</option>
</field>
Expand All @@ -23,7 +22,6 @@
rows="10"
cols="150"
filter="JComponentHelper::filterText"
required="true"
onchange="tjnotificationsAdmin.notification.validateSmsLength(this)"
/>

Expand Down
2 changes: 0 additions & 2 deletions src/com_tjnotifications/admin/models/forms/webhookfields.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
type="contentlanguage"
label="JFIELD_LANGUAGE_LABEL"
description="COM_TJNOTIFICATIONS_NOTIFICATION_LANGUAGE_DESC"
required="true"
>
<option value="*">JALL</option>
</field>
Expand Down Expand Up @@ -45,7 +44,6 @@
cols="350"
filter="JComponentHelper::filterText"
class="validate-json"
required="true"
/>
</fieldset>
</form>
2 changes: 0 additions & 2 deletions src/com_tjnotifications/admin/models/forms/whatsappfields.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
type="contentlanguage"
label="JFIELD_LANGUAGE_LABEL"
description="COM_TJNOTIFICATIONS_NOTIFICATION_LANGUAGE_DESC"
required="true"
>
<option value="*">JALL</option>
</field>
Expand All @@ -23,7 +22,6 @@
rows="10"
cols="150"
filter="JComponentHelper::filterText"
required="true"
/>
</fieldset>
</form>
121 changes: 91 additions & 30 deletions src/com_tjnotifications/admin/models/notification.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,8 @@ public function updateReplacementTags($data)
*/
public function save($data)
{
$isNew = true;
// Determine if this is a new record or edit
$isNew = empty($data['id']) || $data['id'] == 0;

// 1 - save template first
if (!empty($data))
Expand All @@ -345,14 +346,22 @@ public function save($data)

if (!parent::save($data))
{
// Log the error for debugging
Factory::getApplication()->getLogger()->error('TJNotifications: Failed to save template - ' . $this->getError());
return false;
}
else
{
$db = Factory::getDbo();
$db = Factory::getDbo();

// Get the template ID - use insertid() for new records, or the existing ID for edits
if ($isNew) {
$templateId = $db->insertid();
} else {
$templateId = $data['id'];
}

// IMPORTANT to set new id in state, it is fetched in controller later
// Get current Template id
$templateId = $db->insertid();
$this->setState('com_tjnotifications.edit.notification.id', $templateId);
$this->setState('com_tjnotifications.edit.notification.new', $isNew);
}
Expand Down Expand Up @@ -397,25 +406,58 @@ public function save($data)
// 2.2 Find existing template config entries to be deleted (i.e. language specific templates removed by user)
foreach ($data[$backend][$backend . 'fields'] as $backendName => $backendFieldValues)
{
// Webhook stuff starts here
if ($backend == 'webhook' && $data[$backend]['state'])
// Backend-specific validation - only validate when backend is enabled
if ($data[$backend]['state'])
{
// If not using global webhook URLs & custom webhooks URLs are also empty
if (empty($backendFieldValues['use_global_webhook_url']) && empty($backendFieldValues['webhook_url']))
// Common validation for all backends
if (empty($backendFieldValues['language']))
{
$this->setError(Text::_('COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_CUSTOM_WEBHOOK_URLS'));
$errorKey = 'COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_' . strtoupper($backend) . '_LANGUAGE_REQUIRED';
$this->setError(Text::_($errorKey));

return false;
}
// If using global webhook URL & the global URLs are empty
elseif ($backendFieldValues['use_global_webhook_url'] && empty($webhookUrls[0]))

// Check if body exists and is not empty (trim to handle whitespace)
if (!isset($backendFieldValues['body']) || trim($backendFieldValues['body']) === '')
{
$this->setError(Text::_('COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_GLOBAL_WEBHOOK_URLS'));
$errorKey = 'COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_' . strtoupper($backend) . '_BODY_REQUIRED';
$this->setError(Text::_($errorKey));

return false;
}

// Email-specific validation
if ($backend == 'email')
{
if (empty($backendFieldValues['subject']))
{
$this->setError(Text::_('COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_EMAIL_SUBJECT_REQUIRED'));

return false;
}
}

// Webhook-specific validation
if ($backend == 'webhook')
{
// If not using global webhook URLs & custom webhooks URLs are also empty
if (empty($backendFieldValues['use_global_webhook_url']) && empty($backendFieldValues['webhook_url']))
{
$this->setError(Text::_('COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_CUSTOM_WEBHOOK_URLS'));

return false;
}
// If using global webhook URL & the global URLs are empty
elseif (!empty($backendFieldValues['use_global_webhook_url']) && empty($webhookUrls[0]))
{
$this->setError(Text::_('COM_TJNOTIFICATIONS_TEMPLATE_ERR_MSG_GLOBAL_WEBHOOK_URLS'));

return false;
}
}
}
// Webhook stuff ends here
// Backend validation ends here

// Iterate through each lang. specific config entry
foreach ($existingBackendConfigs as $existingBackendConfig)
Expand Down Expand Up @@ -453,23 +495,40 @@ public function save($data)
// Function call to delete template configs
$this->deleteBackendConfigs($backendConfigIdsToBeDeleted);

// 2.3 Common data for saving
$createdOn = !empty($data['created_on']) ? $data['created_on'] : '';
$updatedOn = !empty($data['updated_on']) ? $data['updated_on'] : '';

// 2.4 try saving all backend specific configs
// This has repeatable data eg: $data['email']['emailfields'] or $data['sms']['smsfields']
foreach ($data[$backend][$backend . 'fields'] as $backendName => $backendFieldValues)
{
$templateConfigTable = Table::getInstance('Template', 'TjnotificationTable', array('dbo', $db));
$templateConfigTable->load(array('template_id' => $templateId, 'backend' => $backendName));

// Determine if this is a new backend config or existing one
$isNewBackendConfig = empty($backendFieldValues['id']);

// Load existing record if ID is present (edit mode)
if (!$isNewBackendConfig)
{
$templateConfigTable->load($backendFieldValues['id']);
}

// Non-repeat data
$templateConfigTable->template_id = $templateId;
$templateConfigTable->backend = $backend;
$templateConfigTable->state = $data[$backend]['state'];
$templateConfigTable->created_on = $createdOn;
$templateConfigTable->updated_on = $updatedOn;

// Set datetime fields properly
$date = Factory::getDate();
if ($isNewBackendConfig)
{
// New backend config - set created_on to now
$templateConfigTable->created_on = $date->toSql(true);
$templateConfigTable->updated_on = $db->getNullDate();
}
else
{
// Existing backend config - keep created_on, update updated_on
// created_on is already loaded from database
$templateConfigTable->updated_on = $date->toSql(true);
}

// Get params data
// State, emailfields / smsfields
Expand All @@ -487,9 +546,9 @@ public function save($data)
$templateConfigTable->params = json_encode($params);

// Repeatable data
$templateConfigTable->subject = !empty($backendFieldValues['subject']) ? $backendFieldValues['subject']: '';
$templateConfigTable->body = $backendFieldValues['body'];
$templateConfigTable->language = $backendFieldValues['language'];
$templateConfigTable->subject = !empty($backendFieldValues['subject']) ? $backendFieldValues['subject'] : '';
$templateConfigTable->body = isset($backendFieldValues['body']) ? $backendFieldValues['body'] : '';
$templateConfigTable->language = isset($backendFieldValues['language']) ? $backendFieldValues['language'] : '*';
$templateConfigTable->is_override = 0;

// Webhook stuff starts here
Expand All @@ -504,15 +563,17 @@ public function save($data)
$templateConfigTable->provider_template_id = $backendFieldValues['provider_template_id'];
}

// Save backend in config table
if (empty($backendFieldValues['id']))
// Set ID if exists (for updates), otherwise it will be a new insert
if (!empty($backendFieldValues['id']))
{
$templateConfigTable->save($templateConfigTable);
$templateConfigTable->id = $backendFieldValues['id'];
}
else

// Save backend in config table
if (!$templateConfigTable->store())
{
$templateConfigTable->id = $backendFieldValues['id'];
$templateConfigTable->save($templateConfigTable);
$this->setError($templateConfigTable->getError());
return false;
}
}
}
Expand Down Expand Up @@ -716,7 +777,7 @@ public function updateTemplates($template, $client)
$templateConfigTable->updated_on = '';
$templateConfigTable->is_override = 0;
$templateConfigTable->params = json_encode([]);
$templateConfigTable->save($templateConfigTable);
$templateConfigTable->store();
}
}
}
Expand Down