Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#1454 closed bug (fixed (in master))

Fix spell ordering

Reported by: magnate Owned by: noz
Milestone: 3.3.0 Keywords: magic
Cc:

Description

Change History (6)

comment:1 Changed 7 years ago by nckmccnnll

I think there has been talk of adopting the FA spell system (which allows for different spell book conmtents depending on class, and is also generalisable beyond two realms if you ever want to do that). This seems like a good time (it shouldn't be a big change).

Last edited 7 years ago by nckmccnnll (previous) (diff)

comment:2 Changed 7 years ago by magnate

Hmmm. If we are moving towards a code freeze for 3.3, I would suggest this is not a good time. We revert to the old behaviour for 3.3, and refactor/redesign for 3.4.

But I may have overestimated the urgency of moving towards 3.3.

comment:3 Changed 7 years ago by noz

  • Owner set to noz
  • Status changed from new to assigned

Should be sufficient to simply sort the spells by book position in spell_collect_from_book().

comment:4 Changed 7 years ago by noz

  • Status changed from assigned to pending

comment:5 Changed 7 years ago by magnate

[r057a05a]

comment:6 Changed 7 years ago by magnate

  • Resolution set to fixed
  • Status changed from pending to closed

In master as of [r5c8f16e]

Note: See TracTickets for help on using tickets.