Quote:
|
Cavedude or KLS
Another thing I noticed was adding lootdrops either did not let you specify mindrop, droplimit, multiplier, probability or if it did would ignore your input. I understand that PEQ DB pulls everything off of magelo now in terms of the loot but for doing custom loot it was annoying to have to create then edit. Not sure if this is correct but it works and here is a patch against r371. Just noticed r372 is on svn but nothing effected by this patch was touched in r372. Code:
Index: templates/loot/lootdrop.add.tmpl.php |
Could you tell me exactly where the problem is? It seems to be working fine for me.
|
When you click on loot, select the zone and npc.
LootDrops associated with this LootTable: 3 http://127.0.0.1/peqedit/images/add.gif Click on add and in this example lets say I am adding a new lootdrop and I want to set probability to 50. It would save as 0 and I would have to then click on that 0 value which brings up the edit page and enter it there and then it would save. Minor issue but I noticed it earlier when populating loot in a custom zone and wanted to fix it. Personal preference is all. I also changed my copy so when I add items to a loot drop I can set everything on that screen instead of adding a item then having to go in and change stuff like minlvl maxlvl charges and chance. edit: I probably should be pulling from your daily dump more often but its not like anything is really broken and generating patches seems best off the svn revision. |
The daily dump is best for starting new. or never changing. the SVN is good when you have existing and want to see what changed. Also posting diff for changes = awesome
|
Quote:
I've been guilty of that in the past :D |
Hmm, the only option I see broken is when adding an existing lootdrop from search. The options to create a new lootdrop or add an existing one by ID both work fine for me.
|
Hmm looking at lootdrop.add.tmpl.php I see how it is written for default values of 0,0,1,100.
I wonder if by putting anything into those fields instead of leaving them blank was what caused it to create lootdrops with 0,0,0,0 in the fields. Since that patch works I'm going to be lazy and not revert and check but that might be why I was seeing zero across the board. |
Oh, I wanted to mention that. Your changes to lootdrop.add.tmpl.php should be reverted. You replaced the default values (which exist only to save on typing for the user) to variables that aren't even defined yet. The input field itself is what creates those variables, so it's referencing a variable it hasn't even submitted yet. PHP is very forgiving of course, but another language like C++ wouldn't like that too much and crash.
This change creates NULLs for the input boxes (since the variable doesn't exist yet), so if you clicked submit without filling out all the boxes the DB would fill with default values, which are 0s. If that's not the problem, then there may be something else going on for you else where. |
Quote:
I did take a look at \lib\loot.php and saw in function assign_lootdrop that droplimit, mindrop, multiplier and probability are defined which is why I made the changes to lootdrop.add.tmpl.php |
Yep, lootdrop.add.tmpl.php creates those variables and sends them to loot.php for use in assign_lootdrop().
|
Well I feel stupid to an extent. When I woke up today I deleted every file with a change date that did not match up with the SVN, then reapplied my custom changes for misc stuff like website integration and everything works like it should.
So at some point in time I changed one of the pages and was not careful with copying from SVN to my web folder and that would explain when I was adding loot drops mindrop,droplimit,multiplier,probability were zero and also I just checked adding items and noticed that I was not careful in the past because the current revision from svn when I add items now I do have the option to change percent. So thanks cavedude for suffering my posts. It was still nice to take a few min and look at how everything is setup and play with it. I was able to make some personalized changes to aspects of the tool yesterday and get an idea for how it should look based on the current templates. Anyways, thanks as always to you and everyone else from peq that keep this tool up to date. |
Reference: http://www.peqtgc.com/phpBB3/viewtopic.php?f=29&t=14068
I think this is a good idea and decided to go mucking around inside the editor... again. This time I am going to include some pictures of the table I tested against. http://imageshack.us/a/img802/960/zlzo.png Then what the new option you clicked on brings up. I used one of the 13x13 icons included with the editor for this test. http://imageshack.us/a/img5/4364/m2ue.png And what the result ended up being after choosing the copy option. http://imageshack.us/a/img4/2701/mrqo.png You can either choose move (move and delete from old) or copy (move and retain original). Here is a patch and I'm pretty sure I did not fubar up this time but its probably not the best way to handle it. Either way you cut it, it does work and can be made pretty as needed. IE I created a new template page but was that really the best method? Code:
Index: lib/loot.php |
Thanks for that! I'll be merging it into the next commit, it's very well done. The only thing I changed was I made copy the default option.
|
Also, I wanted to say keep mucking! Once you get a solid grasp of the editor, I'll have no problem giving you SVN access, so you can commit your changes directly.
|
All times are GMT -4. The time now is 02:43 PM. |
Powered by vBulletin®, Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.