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

Smarty notices for New Activity / New Case for tags for attachments #20957

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

demeritcowboy
Copy link
Contributor

Overview

  1. Don't create any tags for attachments.
  2. Create a regular activity, or create a new case.
  3. Smarty notices - the exact set of notices depends on whether you have any tags for other types or not.

Before

Something like

    Notice: Undefined index: tag_1 in include() (line 60 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tagsetInfo in include() (line 68 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tag_2 in include() (line 116 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tag_2 in include() (line 118 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tagsetInfo in include() (line 121 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tag_3 in include() (line 116 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tag_3 in include() (line 118 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tagsetInfo in include() (line 121 of .../templates_c/en_US/%%3F/3F9/3F9B0555%%attachment.tpl.php).
    Notice: Undefined index: tag in include() (line 108 of .../templates_c/en_US/%%E8/E8E/E8EBFB02%%Case.tpl.php).

After

There are still some others depending on your config, but this covers the stock install for attachments.

Technical Details

They can be legitimately missing because it's reasonable not to display a tag select widget if you don't have any tags for the type defined.

Comments

@civibot
Copy link

civibot bot commented Jul 26, 2021

(Standard links)

@civibot civibot bot added the master label Jul 26, 2021
@eileenmcnaughton
Copy link
Contributor

thanks @demeritcowboy

@eileenmcnaughton eileenmcnaughton merged commit 403b032 into civicrm:master Jul 27, 2021
@demeritcowboy demeritcowboy deleted the smarty-tags branch July 27, 2021 12:58
@demeritcowboy
Copy link
Contributor Author

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants