Issue information

Issue ID
#1140
Status
Fixed
Severity
None
Started
Hercules Elf Bot
Mar 9, 2008 14:27
Last Post
Hercules Elf Bot
Mar 9, 2008 14:27
Confirmation
N/A

Hercules Elf Bot - Mar 9, 2008 14:27

Originally posted by [b]theultramage[/b]
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=1140

As people may have noticed, the mapserver currently supports two modes of saving mapserver script variables: mapreg.txt and `mapreg` table.
The thing I find hard to understand is why an SQL build still saves data into a plaintext file, and actually needs a compilation setting to switch the behavior.

I propose to do the following:
  1. Write a mapreg.txt -> `mapreg` converter, which would parse the mapreg.txt file and dump the data into the table (like other converters do).
    I have already written a primitive (non-escaping, non-inserting) version - mapreg-convert.cpp.
  2. Replace uses of the compilation option with a classic TXT_ONLY define.
  3. Post a critical compatibility breakage notice, with steps how to make the transition from this TXT/SQL hybrid to pure SQL.