This idea just poped in my head while i was updating our server files to freya.

When i looked at the multisells, it was a total mess, so many multisells of which you have no idea for what they are, are they custom, or some of l2j's, or modified... etc etc. So i've just decided to make a support for custom multisells. Those custom multisells should be placed at data/multisell/custom  (you have to create such a folder).

Now comes the good part, if there is duplicate multisell id (ex: data/multisell/001.xml and data/multisell/custom/001.xml) the multisell from the custom folder should (in theory, not tested, but it should work) override the multisell loaded from the data/multisell folder. I've even decided to make a comment support for the custom multisells, a space " " seperates the multisell id from the comment (ex: 103 Oly exchange.xml, 1004 - Blacksmith material craft with no recipe requirement.xml).

This way is a lot easier to check what is that multisell for, instead of oppening it, and pray that there will be comments inside, its also easier to locate a multisell for whick you know what it is for, but you dont know the id

UNTESTED!

Index: java/com/l2jserver/gameserver/datatables/MultiSell.java
===================================================================
--- java/com/l2jserver/gameserver/datatables/MultiSell.java    (revision 17)
+++ java/com/l2jserver/gameserver/datatables/MultiSell.java    (working copy)
@@ -63,12 +63,14 @@
    {
        _entries = new TIntObjectHashMap<ListContainer>();
        load();
+        loadCustom();
    }
    
    public final void reload()
    {
        _entries.clear();
        load();
+        loadCustom();
    }
    
    /**
@@ -218,6 +220,44 @@
        _log.config("MultiSell: Loaded " + _entries.size() + " lists.");
    }

+    private final void loadCustom()
+    {
+        Document doc = null;
+        int id = 0;
+        List<File> files = new FastList<File>();
+        hashFiles("multisell/custom", files);
+        
+        for (File f : files)
+        {
+            try
+            {
+                id = Integer.parseInt(f.getName().replaceAll(".xml", "").split(" ")[0]);
+                DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
+                factory.setValidating(false);
+                factory.setIgnoringComments(true);
+                doc = factory.newDocumentBuilder().parse(f);
+            }
+            catch (Exception e)
+            {
+                _log.log(Level.SEVERE, "Error loading custom file " + f, e);
+                continue;
+            }
+            
+            try
+            {
+                ListContainer list = parseDocument(doc);
+                list.setListId(id);
+                _entries.put(id, list);
+            }
+            catch (Exception e)
+            {
+                _log.log(Level.SEVERE, "Error in custom file " + f, e);
+            }
+        }
+        verify();
+        _log.config("MultiSell: Loaded " + _entries.size() + " custom lists.");
+    }
+    
    private final ListContainer parseDocument(Document doc)
    {
        int entryId = 1;

 

Credits: nik from l2jforum

ok that is very usefull :D . works only in freya?

Pretty much useless.

The fact that he had no idea about the multisells, meant that he had to seperate them like that?

 

Link of the share?

UNTESTED !

 

Nice share Man. But Who Test ? :)

Sky" post="1424454" timestamp="1286912321"]

Nice share Man. But Who Test ? :)

 

Untested = Not Tested = Never tested by anybody.

pff, spammer.

It's possible create custom multisell at runtime without xml.

Multisell in java code? :) NICE! Well but I still dont get it how to use it