Jump to content
Sign in to follow this  
Fredi

Support ticket for Phoenix

Recommended Posts

Added Support ticket module in Marketplace

Support ticket for Phoenix module.
Compatibility: Phoenix Tested on OSCOM CE Phoenix 1.0.4.2 PHP 7.3

All credit goes to the original developer: Copyright (c) 2003 Henri Schmidhuber IN-Solution and all other contributers of previous versions.
Many thanks to 
Lee Foster for his great participation in the modernization of this module.

Code corrections have been made to adapt to php 7.3, for Phoenix and for the standard module menu in the Admin. The addon includes 2 languages: English and Russian. Full Package.

Simply copy over the files and install and turn on the new add-on.

Go to Modules -> Dashboard -> Install Module -> Support Tickets -> Install Module
Set options through Modules -> Dashboard -> Support Tickets


Support forum for osCommerce in russian language - from Ashkelon. Support since 2002.

Best regards,

Fredi

Share this post


Link to post
Share on other sites

The module has been updated.


Support forum for osCommerce in russian language - from Ashkelon. Support since 2002.

Best regards,

Fredi

Share this post


Link to post
Share on other sites

Added page ticket_priority.php to Admin menu, and kbnk updates Admin menu to new stile.

 

184739.jpg


Support forum for osCommerce in russian language - from Ashkelon. Support since 2002.

Best regards,

Fredi

Share this post


Link to post
Share on other sites

Updated to 1.0.5.2 and getting multiple errors.

Notice: Undefined index: action in C:\xampp\htdocs\360v3\admin\ticket_admin.php on line 144

Notice: Undefined index: oID in C:\xampp\htdocs\360v3\admin\ticket_admin.php on line 102

Notice: Undefined variable: oInfo in C:\xampp\htdocs\360v3\admin\ticket_admin.php on line 102

Notice: Undefined index: action in C:\xampp\htdocs\360v3\admin\ticket_admin.php on line 102

Notice: Undefined index: action in C:\xampp\htdocs\360v3\admin\ticket_admin.php on line 129

if (((!$_GET['oID']) || ($_GET['oID'] == $ticket_admin['ticket_admin_id'])) && (!$oInfo) && (substr($_GET['action'], 0, 3) != 'new')) { <--- Line 102
      $oInfo = new objectInfo($ticket_admin);
}
                                                                                                                                             
                                                                                                                                             
switch ($_GET['action']) { < --- Line 144      


if (substr($_GET['action'], 0, 3) != 'new') { < --- Line 129

 

 

Share this post


Link to post
Share on other sites
if ((!isset($_GET['oID']) || ($_GET['oID'] == $ticket_admin['ticket_admin_id'])) && (!isset($oInfo)) && (!isset($_GET['action']) || (substr($_GET['action'], 0, 3) != 'new'))) {
      $oInfo = new objectInfo($ticket_admin);
}
                                                                                                                                             
                                                                                                                                             
switch ($_GET['action'] ?? null) {    


if (!isset($_GET['action']) || (substr($_GET['action'], 0, 3) != 'new')) {

Or just do

$action = $_GET['action'] ?? null;

at the top and replace all the $_GET['action'] with $action. 


Always back up before making changes.

Share this post


Link to post
Share on other sites

Ok most errors fixed now on to -

Notice: Undefined variable: ticket_last_used_admin in C:\xampp\htdocs\360v3\admin\ticket_view.php on line 177

Line 177 is - 

echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, ($ticket_last_used_admin ? $ticket_last_used_admins : TICKET_DEFAULT_ADMIN_ID) ) . "<br><br>";

 

Share this post


Link to post
Share on other sites

Probably

echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, ($ticket_last_used_admin ?? TICKET_DEFAULT_ADMIN_ID) ) . "<br><br>";

But it's possible that it would need to be something like

echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, (($ticket_last_used_admin ?? false) ?: TICKET_DEFAULT_ADMIN_ID) ) . "<br><br>";

It really depends how $ticket_last_used_admin is set. 


Always back up before making changes.

Share this post


Link to post
Share on other sites
9 hours ago, ecartz said:

Probably


echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, ($ticket_last_used_admin ?? TICKET_DEFAULT_ADMIN_ID) ) . "<br><br>";

But it's possible that it would need to be something like


echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, (($ticket_last_used_admin ?? false) ?: TICKET_DEFAULT_ADMIN_ID) ) . "<br><br>";

It really depends how $ticket_last_used_admin is set. 

That's the poblem, I can't find that it is set anywhere.

Share this post


Link to post
Share on other sites
3 hours ago, LeeFoster said:

That's the problem, I can't find that it is set anywhere.

If it's never set, then the first one or

echo '  ' . TEXT_ADMIN . '&nbsp;' . tep_draw_pull_down_menu('ticket_admin', $ticket_admins, TICKET_DEFAULT_ADMIN_ID ) . "<br><br>";

would give the same effect. 


Always back up before making changes.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

×