Merge pull request #8 from geojak/master

make random stack sizes optional

README_Geojak.md

@@ -1,7 +1,7 @@
 1. Added auto bid feature, doubled the existing auto buy code and slighly editted
 2. Added auto pricing heuristic, these are activate when posting an item with bid price 0, the buyout price works as a minimum
 3. Added a keybind for the post button, this significantly speeds up mass posting of items when you dont need to change saved prices
-4. Changed the default post stack size to rand(1,max) instead of always max
+4. New option to change the default post stack size to rand(1,max) instead of always max stacks, use /aux post stack to toggle
 5. FIXED TWILLIGHT CULTIST SET NOT DISENCHATABLE
 6. use SHagu tweaks vendor values if aux values not avaialbe (might need fixing if you dont actually have shaguweaks)
 7. dont auto bid on own items
@@ -9,4 +9,5 @@
 9. enchanting created wands are not encahntabke, same with twillight stuff at 5. reworked implementation for easier addition
 10. disenchant values are calcualted from min(value,value_today), this protects users from low enchanting mat prices
 11. advanced trade skill menu compabillity to display aux crafting cost
-12. changed default post duration from 8 to 24, since turtle wow got no fees anyway
+12. changed default post duration from 8 to 24, since turtle wow got no fees anyway
+13. new filter "isgear" which can be used to generic filter out all white and green armor and weapons that have incosinstent market value

aux-addon.lua

@@ -59,6 +59,7 @@ function handle.LOAD()
         crafting_cost = true,
         post_bid = false,
         post_duration = post.DURATION_24,
+		post_stack = true,
         items = {},
         item_ids = {},
         auctionable_items = {},

core/slash.lua

@@ -21,6 +21,9 @@ function SlashCmdList.AUX(command)
     elseif arguments[1] == 'ignore' and arguments[2] == 'owner' then
 	    aux.account_data.ignore_owner = not aux.account_data.ignore_owner
         aux.print('ignore owner ' .. status(aux.account_data.ignore_owner))
+	elseif arguments[1] == 'post' and arguments[2] == 'stack' then
+        aux.account_data.post_stack = not aux.account_data.post_stack
+	    aux.print('post stack ' .. status(aux.account_data.post_stack))
     elseif arguments[1] == 'post' and arguments[2] == 'bid' then
         aux.account_data.post_bid = not aux.account_data.post_bid
 	    aux.print('post bid ' .. status(aux.account_data.post_bid))
@@ -61,6 +64,7 @@ function SlashCmdList.AUX(command)
 		aux.print('- ignore owner [' .. status(aux.account_data.ignore_owner) .. ']')
 		aux.print('- post bid [' .. status(aux.account_data.post_bid) .. ']')
         aux.print('- post duration [' .. aux.color.blue(aux.account_data.post_duration / 60 .. 'h') .. ']')
+		aux.print('- post stack [' .. status(aux.account_data.post_stack) .. ']')
         aux.print('- crafting cost [' .. status(aux.account_data.crafting_cost) .. ']')
 		aux.print('- tooltip value [' .. status(tooltip_settings.value) .. ']')
 		aux.print('- tooltip daily [' .. status(tooltip_settings.daily) .. ']')

core/tooltip.lua

@@ -155,10 +155,12 @@ function game_tooltip_hooks:SetInboxItem(index)
     local name, _, quantity = GetInboxItem(index)
     local id = name and info.item_id(name)
     if id then
-        local _, itemstring, quality = GetItemInfo(id)
+        local _, itemstring, quality =  (id)
-        local hex = aux.select(4, GetItemQualityColor(tonumber(quality)))
+		if quality and itemstring then
-        local link = hex ..  '|H' .. itemstring .. '|h[' .. name .. ']|h' .. FONT_COLOR_CODE_CLOSE
+			local hex = aux.select(4, GetItemQualityColor(tonumber(quality)))
-        extend_tooltip(GameTooltip, link, quantity)
+			local link = hex ..  '|H' .. itemstring .. '|h[' .. name .. ']|h' .. FONT_COLOR_CODE_CLOSE
+			extend_tooltip(GameTooltip, link, quantity)
+		end
     end
 end
 

tabs/post/core.lua

@@ -431,23 +431,28 @@ function update_item(item)
     UIDropDownMenu_SetSelectedValue(duration_dropdown, settings.duration)
 
     hide_checkbox:SetChecked(settings.hidden)
+	
 	local ii = 1
-    if selected_item.max_charges then
+	if selected_item.max_charges then
-	    for i = selected_item.max_charges, 1, -1 do
+		for i = selected_item.max_charges, 1, -1 do
 			if selected_item.availability[i] > 0 then
 				stack_size_slider:SetMinMaxValues(1, i)
 				ii=i
 				break
 			end
-	    end
+		end
-    else
+	else
 		ii = min(selected_item.max_stack, selected_item.aux_quantity)
-	    stack_size_slider:SetMinMaxValues(1, min(selected_item.max_stack, selected_item.aux_quantity))
+		stack_size_slider:SetMinMaxValues(1, min(selected_item.max_stack, selected_item.aux_quantity))
-    end
+	end
-	
+		
-	stack_size_slider:SetValue(math.random(1,ii))
+	if not aux.account_data.post_stack then	
-    --stack_size_slider:SetValue(aux.huge)
+		stack_size_slider:SetValue(math.random(1,ii))
-	--stack_size_slider:SetValue(1)
+	else
+		stack_size_slider:SetValue(aux.huge)
+		--stack_size_slider:SetValue(1)
+	end
+
     quantity_update(true)
 
     unit_start_price_input:SetText(money.to_string(settings.start_price, true, nil, nil, true))

tabs/search/filter.lua

@@ -323,7 +323,7 @@ function set_filter_display_offset(x_offset, y_offset)
 end
 
 function initialize_filter_dropdown()
-	for _, filter in ipairs(T.temp-T.list('and', 'or', 'not', 'price', 'profit', 'vendor-profit', 'disenchant-profit', 'percent', 'disenchant-percent', 'bid-price', 'bid-profit', 'bid-vendor-profit', 'bid-disenchant-profit', 'bid-percent', 'bid-disenchant-percent', 'item', 'tooltip', 'min-level', 'max-level', 'rarity', 'left', 'utilizable', 'seller')) do
+	for _, filter in ipairs(T.temp-T.list('and', 'or', 'not', 'price', 'profit', 'vendor-profit', 'disenchant-profit', 'percent', 'disenchant-percent', 'bid-price', 'bid-profit', 'bid-vendor-profit', 'bid-disenchant-profit', 'bid-percent', 'bid-disenchant-percent', 'item', 'tooltip', 'min-level', 'max-level', 'rarity', 'left', 'utilizable', 'seller', 'isgear')) do
 		UIDropDownMenu_AddButton(T.map(
 			'text', filter,
 			'value', filter,

util/filter.lua

@@ -38,7 +38,7 @@ M.filters = {
             return default_filter(str).validator()
         end,
     },
-
+	
     ['item'] = {
         input_type = 'string',
         validator = function(name)
@@ -229,6 +229,16 @@ M.filters = {
             end
         end
     },
+	
+	['isgear'] = {
+		input_type = '',
+        validator = function()
+            return function(auction_record)
+                return not(info.item(auction_record.item_id).slot == '')
+            end
+        end,
+	    },
+	
 }
 
 function operator(str)