Code clenaup

Re-arranged the order of some of the final debug entries, and added a
condition test for the presence of a <star> tag before performing a
string replacement.

Author: Dave-Morton

chatbot/conversation_start.php

@@ -160,20 +160,15 @@
     $time_start = $convoArr['time_start'];
     $time_end = microtime(true);
     $time = round(($time_end - $time_start) * 1000,4);
-    runDebug(__FILE__, __FUNCTION__, __LINE__, "Conversation Ending. Elapsed time: $time milliseconds.", 0);
-    runDebug(__FILE__, __FUNCTION__, __LINE__, "FINAL CONVO ARRAY",4);
-    $final_convoArr = $convoArr;
-    unset($final_convoArr['nounList']);
-    runDebug(__FILE__, __FUNCTION__, __LINE__, print_r($final_convoArr,true), 4);
     unset($convoArr['nounList']);
+    $final_convoArr = $convoArr;
   }
   else
   {
     runDebug(__FILE__, __FUNCTION__, __LINE__, "Conversation intialised waiting user", 2);
   }
-  #if ($display == '') $display = $convoArr['send_to_user'];
   runDebug(__FILE__, __FUNCTION__, __LINE__, "Closing Database", 2);
   db_close($con);
-  //save_file(_DEBUG_PATH_ . 'function_list.txt', print_r(get_defined_functions()['user'], true));
   display_conversation($convoArr);
+  runDebug(__FILE__, __FUNCTION__, __LINE__, "Conversation Ending. Elapsed time: $time milliseconds.", 0);
   $convoArr = handleDebug($convoArr); // Make sure this is the last line in the file, so that all debug entries are captured.

chatbot/core/aiml/parse_aiml_as_XML.php

@@ -568,8 +568,8 @@ function parse_html_tag($convoArr, $element, $parentName, $level)
     $response_string = $element->asXML();
     $response_string = str_replace('<text>', '', $response_string);
     $response_string = str_replace('</text>', '', $response_string);
-    $star = $convoArr['star'][1];
-    $response_string = str_replace('<star/>', $star, $response_string);
+    $star = (isset($convoArr['star'][1])) ? $convoArr['star'][1] : '';
+    if ($star != '') $response_string = str_replace('<star/>', $star, $response_string);
     return $response_string;
   }