Not a Bug Question about drawing kit

[Levis]

I think @Jack Rackham added this:

if(itm1 == "drawing_kit")
            {
                if(CheckAttribute(mchref,"quest.drawing_kit"))
                {
                    switch(mchref.quest.drawing_kit)
                    {
                        case "speed":
                            if(CheckCharacterPerk(mchref, "SailingProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "SandbankManeuver"))
                                {
                                    DeleteAttribute(mchref,"perks.list.SandbankManeuver");
                                }
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "Turn180"))
                                {
                                    DeleteAttribute(mchref,"perks.list.Turn180");
                                }
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.ShipSpeedUp");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "maneuver":
                            if(CheckCharacterPerk(mchref, "SailingProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "SandbankManeuver"))
                                {
                                    DeleteAttribute(mchref,"perks.list.SandbankManeuver");
                                }
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "Turn180"))
                                {
                                    DeleteAttribute(mchref,"perks.list.Turn180");
                                }
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.ShipTurnRateUp");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "turn180":
                            if(CheckCharacterPerk(mchref, "SailingProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "SandbankManeuver"))
                                {
                                    DeleteAttribute(mchref,"perks.list.SandbankManeuver");
                                }
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.Turn180");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "sandbank":
                            if(CheckCharacterPerk(mchref, "SailingProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.SandbankManeuver");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "storm":
                            if(CheckCharacterPerk(mchref, "SailingProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.StormProfessional");
                            }
                                mchref.quest.drawing_kit = "none";
                        break;

                        case "seawolf":
                            DeleteAttribute(mchref,"perks.list.SailingProfessional");
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "def1":
                            if(CheckCharacterPerk(mchref, "ShipDefenceProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.ShipDefenceProfessional");
                            }
                            else
                            {
                                if(CheckCharacterPerk(mchref, "AdvancedBattleState"))
                                {
                                    DeleteAttribute(mchref,"perks.list.AdvancedBattleState");
                                }
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.BasicBattleState");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "def2":
                            if(CheckCharacterPerk(mchref, "ShipDefenceProfessional"))
                            {
                                DeleteAttribute(mchref,"perks.list.ShipDefenceProfessional");
                            }
                            else
                            {
                                DeleteAttribute(mchref,"perks.list.AdvancedBattleState");
                            }
                            mchref.quest.drawing_kit = "none";
                        break;

                        case "def3":
                            DeleteAttribute(mchref,"perks.list.ShipDefenceProfessional");
                            mchref.quest.drawing_kit = "none";
                        break;
                    }
                }
            }

Should this only remove the perk or should it also add perkpoints back to the character?

 

[Pieter Boelen]

Whatever it does, I think it is exclusive to his storyline(s).

 

[Levis]

well I noticed perks are being removed and added by this item but the perkcost isn't taken into account here so I wanted to be sure that wasn't a problem here. There are functions now which can add/remove perks from a character. for now I didn't change this.

 

[Jack Rackham]

I don't think there existed any perkpoints when I added this item.

 

[Pieter Boelen]

I don't think there existed any perkpoints when I added this item.

But those have existed since the stock game? It was just always 1 per perk.

 

[Levis]

I don't think there existed any perkpoints when I added this item.

Should the removal of the perk give back points to the player or doesn't it matter here?

 

[Jack Rackham]

I don't even remember what's going on there. Have to check first.

 

[Jack Rackham]

Should the removal of the perk give back points to the player or doesn't it matter here?

I don't think it matters.

 

[Levis]

ok. then I'm going to close the topic now. It always worked so I guess no need to change it .