Skip to content

Commit a310e83

Browse files
authored
Merge pull request #3394 from vegaprotocol/bugfix/3393-nil-pointer-liqidity-params
us default when instanciating markete if LiquidityMonitoringParameters is nil OR LiquidityMonitoringParameters.TargetStakeParameters is nil
2 parents dfbbc33 + 11d5b12 commit a310e83

1 file changed

Lines changed: 14 additions & 7 deletions

File tree

governance/market.go

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -163,19 +163,26 @@ func createMarket(
163163
definition.Changes.PriceMonitoringParameters = pmParams
164164
}
165165

166-
if definition.Changes.LiquidityMonitoringParameters == nil {
166+
if definition.Changes.LiquidityMonitoringParameters == nil ||
167+
definition.Changes.LiquidityMonitoringParameters.TargetStakeParameters == nil {
167168
// get target stake parameters
168169
tsTimeWindow, _ := netp.GetDuration(netparams.MarketTargetStakeTimeWindow)
169170
tsScalingFactor, _ := netp.GetFloat(netparams.MarketTargetStakeScalingFactor)
170171
//get triggering ratio
171172
triggeringRatio, _ := netp.GetFloat(netparams.MarketLiquidityTargetStakeTriggeringRatio)
172173

173-
definition.Changes.LiquidityMonitoringParameters = &types.LiquidityMonitoringParameters{
174-
TargetStakeParameters: &types.TargetStakeParameters{
175-
TimeWindow: int64(tsTimeWindow.Seconds()),
176-
ScalingFactor: tsScalingFactor,
177-
},
178-
TriggeringRatio: triggeringRatio,
174+
params := &types.TargetStakeParameters{
175+
TimeWindow: int64(tsTimeWindow.Seconds()),
176+
ScalingFactor: tsScalingFactor,
177+
}
178+
179+
if definition.Changes.LiquidityMonitoringParameters == nil {
180+
definition.Changes.LiquidityMonitoringParameters = &types.LiquidityMonitoringParameters{
181+
TargetStakeParameters: params,
182+
TriggeringRatio: triggeringRatio,
183+
}
184+
} else {
185+
definition.Changes.LiquidityMonitoringParameters.TargetStakeParameters = params
179186
}
180187
}
181188

0 commit comments

Comments
 (0)