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

Fixes XXXX ticketid error #872

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Fixes XXXX ticketid error #872

wants to merge 1 commit into from

Conversation

clonemeagain
Copy link
Contributor

Forum post: http://www.osticket.com/forum/discussion/74564/ticket-numbers-showing-xxxxxxx/p1

Still a problem, but now its due to the template itself having a variable that it shouldn't.

Suggest SQL patch of same for 1.7.5 upgrade to remove message from web-created-tickets.

Could possibly create admin option to display extId, but would have to modify /open.php and remove the ${ticket.extId} from the array and change the fill to 2.. tested, works. This seems by design, so reworked my solution to simply remove the errant variable.

Workaround for already installed versions: As admin, visit: /scp/pages.php?id=2

Change the text to read:

%{ticket.name},
Thank you for contacting us.

A support ticket has been created and a representative will be getting back to you shortly if necessary.

Please check your email for login information.

Support Team

Forum post: http://www.osticket.com/forum/discussion/74564/ticket-numbers-showing-xxxxxxx/p1

Still a problem, but now its due to the template itself having a variable that it shouldn't.
@greezybacon
Copy link
Contributor

The install script isn't so easily edited. If you want to migrate 1.7 users to 1.8 and drop the confusing text and tokens, then the script should go in the 1.8 repo under include/upgrader/streams/core/16fcef4a-d51f303a.patch.sql

@clonemeagain
Copy link
Contributor Author

Problem doesn't exist in 1.8, but with a impending 1.7.5 it seemed prudent to get this in there. Might have to wait till after Christmas.

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