collapse collapse
* User Info
 
 
Welcome, Guest. Please login or register.
* Search

* Board Stats
  • stats Total Members: 989
  • stats Total Posts: 18363
  • stats Total Topics: 2500
  • stats Total Categories: 7
  • stats Total Boards: 35
  • stats Most Online: 1144

Author Topic: modules/aoc/craftclass.php - Changes  (Read 2274 times)

0 Members and 1 Guest are viewing this topic.

Offline Getrix

  • Contributor
  • *******
  • Posts: 509
  • Karma: +0/-0
modules/aoc/craftclass.php - Changes
« 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:
Code: [Select]
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.";
Success is not final, failure is not fatal: it is the courage to continue that counts.
- Sorry, i dont have time to reply question on PM. Make a topic.

Offline Temar

  • Contributor
  • *******
  • Posts: 1140
  • Karma: +0/-0
    • AoFiles
Re: modules/aoc/craftclass.php - Changes
« Reply #1 on: December 21, 2008, 06:17:28 am »
Commited

just 1 note about the change, Tables dont update them selfs! :p

Offline Getrix

  • Contributor
  • *******
  • Posts: 509
  • Karma: +0/-0
Re: modules/aoc/craftclass.php - Changes
« Reply #2 on: December 25, 2008, 05:05:19 pm »
Quote
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:
Quote
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)
« Last Edit: December 25, 2008, 05:08:02 pm by gerborg »
Success is not final, failure is not fatal: it is the courage to continue that counts.
- Sorry, i dont have time to reply question on PM. Make a topic.

Offline Temar

  • Contributor
  • *******
  • Posts: 1140
  • Karma: +0/-0
    • AoFiles
Re: modules/aoc/craftclass.php - Changes
« Reply #3 on: December 25, 2008, 08:20:13 pm »
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

 

* Recent Posts
[AoC] special char for items module by bitnykk
[February 09, 2024, 09:41:18 pm]


0.8.x updates for AoC by bitnykk
[January 30, 2024, 11:16:08 pm]


0.8.x updates for AO by bitnykk
[January 30, 2024, 11:15:37 pm]


BeBot still alive & kicking ! by bitnykk
[December 17, 2023, 12:58:44 am]


Bebot and Rasberry by bitnykk
[November 29, 2023, 11:04:14 pm]

* Who's Online
  • Dot Guests: 417
  • Dot Hidden: 0
  • Dot Users: 0

There aren't any users online.
* Forum Staff
bitnykk admin bitnykk
Administrator
Khalem admin Khalem
Administrator
WeZoN gmod WeZoN
Global Moderator
SimplePortal 2.3.7 © 2008-2024, SimplePortal