Hiblet/NewZopa01 — pull request #36 by Hiblet (2014-04-03 12:19:46+00:00)
Title: Fixed shortcutting bug effecting removal of orders from OrderRegistry; Fixed Jquery .prop() fn mis-use bug when cancelling EffectiveFrom orders

CHANGED: Market.cs
-- Added logging to drop routine to make drops visible.
-- Added order registry to diagnostic dump routine.

CHANGED: MarketManager.cs
-- Added dump of market before match attempt.
-- Added logging to show that a match attempt is being made for this market.
-- Removed verbose subscription logging.

CHANGED: MarketPriceLevel.cs
-- Remove() now returns false only if the attempt to remove fails.

FIXED: Order.cs
-- PushPricesToNearestTick() avoids possible exposure to shortcut bug.
-- The pushStopPxToNearestTick() fn would not action if pushLimitPxToNearestTick()
-- returned true, so stop price would not be fixed.

FIXED: OrderContainers.cs
-- Order was not being removed from both constituent containers, due to
-- the shortcutting of the OR logic when the first remove succeeded.
-- Now changed to action both removes only if the order is contained, and
-- to only return false if the order remove failed from either.

FIXED: Orders.js
-- Display bad server responses as errors for better diags.
-- Added try-catch-finally around response processing to ensure resubscription.
-- Corrected use of JQuery .prop() function for EffectiveTime scrape.
-- Corrected reference for Intellisense.

CHANGED: PriceSubscriptionManager.cs
-- Refactored subscribe() so that possible exposure to shortcut bug removed.