BeBot - An Anarchy Online and Age Of Conan chat automaton
General => Feedback and Suggestions => Topic started by: Getrix on December 20, 2008, 10:11:15 pm
-
As many players are making alts, and we all know that professions is time consuming todo someone may not want to lvl their alts with it.
Current craft version do not support "None" professions and will keep spam the player with "Set your craft" etc..
My suggestion is to add "None" as a profession
File with changes:
http://dump.sjef.biz/aoc/bebot/craftclasses.phps
Diffs:
66 ORG class1 enum('Alchemist','Architect','Armorsmith','Gemcutter','Weaponsmith') NOT NULL,
66 NEW class1 enum('Alchemist','Architect','Armorsmith','Gemcutter','Weaponsmith','None') NOT NULL,
67 ORG class2 enum('Alchemist','Architect','Armorsmith','Gemcutter','Weaponsmith') NOT NULL,
67 NEW class2 enum('Alchemist','Architect','Armorsmith','Gemcutter','Weaponsmith','None') NOT NULL,
82 ORG $this -> help['command']['setcraft [class1] [class2]']="Sets the two crafting classes for you. Classes can be Alchemist, Architect, Armorsmith, Gemcutter and Weaponsmith";
82 NEW $this -> help['command']['setcraft [class1] [class2]']="Sets the two crafting classes for you. Classes can be Alchemist, Architect, Armorsmith, Gemcutter, Weaponsmith and None";
96 ORG $craftclass = array("Alchemist", "Architect", "Armorsmith", "Gemcutter", "Weaponsmith", "None");
96 NEW $craftclass = array("Alchemist", "Architect", "Armorsmith", "Gemcutter", "Weaponsmith", "None");
112 ORG $output = "Classes can ONLY be Alchemist, Architect, Armorsmith, Gemcutter and Weaponsmith and you MUST set both at the same time.";
112 NEW $output = "Classes can ONLY be Alchemist, Architect, Armorsmith, Gemcutter, Weaponsmith and None. You MUST set both at the same time.";
124 ORG $output = "You have no crafting information set. Please use '/tell <botname> <pre>setcraft [class1] [class2]'. Classes can be Alchemist, Architect, Armorsmith, Gemcutter and Weaponsmith.";
124 NEW $output = "You have no crafting information set. Please use '/tell <botname> <pre>setcraft [class1] [class2]'. Classes can be Alchemist, Architect, Armorsmith, Gemcutter, Weaponsmith and None.";
138 ORG $msg = "You have no crafting information set and you are above level 40. Please use '/tell <botname> <pre>setcraft [class1] [class2]'. Classes can be Alchemist, Architect, Armorsmith, Gemcutter and Weaponsmith. If you havn't picked crafting classes yet this may be the time to do it.";
138 NEW $msg = "You have no crafting information set and you are above level 40. Please use '/tell <botname> <pre>setcraft [class1] [class2]'. Classes can be Alchemist, Architect, Armorsmith, Gemcutter, Weaponsmith and None. If you havn't picked crafting classes yet this may be the time to do it.";
-
Commited
just 1 note about the change, Tables dont update them selfs! :p
-
Botname [2008-12-25 15:50:24] [BUDDY] [BUDDY-ADD] player1
MySQL error (# 0) on query: SELECT * FROM craftingclass WHERE name = 'player1'
Table 'db_bebot.craftingclass' doesn't exist
Bugfix:
Sources/Bot.php:
981 ORG $lookup = $this -> db -> select("SELECT * FROM craftingclass WHERE name = '" . $user . "'", MYSQL_ASSOC);
981 NEW $lookup = $this -> db -> select("SELECT * FROM #___craftingclass WHERE name = '" . $user . "'", MYSQL_ASSOC);
EDIT 1:
We probly need some more settings in the function update_table().
Because earlier the table was called "craftingclass" and its changed to "#___craftingclass" (botname_craftingclass)
-
the update function is only used for updating, new tables E.G. u added a prefix so it will make with prefix shouldnt require an update
Cimmited the Fix to SVN trunk and 0.6