Greeffer's Donchian Channel Strategy

// Greeffer's Donchian Channel Study
// @version=4
//@Greeffer
// S T U D Y
// study(shorttitle="Greeffer's Study", title="Greeffer's  Donchian Channel Study", overlay=true)

// S T R A T E G Y 
strategy(
     "Greeffer's Donchian Channel Strategy",
     "Greeffer's Strategy",
     overlay=true, 
     pyramiding=10,
     default_qty_type=strategy.percent_of_equity,
     default_qty_value=150,
     initial_capital=5000,
     commission_type=strategy.commission.percent,
     commission_value=0.1,
     margin_long=1./30*50, 
     margin_short=1./30*50
     )

// I N P U T S
sma = input(title="sma", type=input.integer, defval=250)
symbol = "BTCUSDT" //symbol = "ETHUSDT"
var upperAmount = 0.001
var lowerAmount = 0.001
crossMiddle = false
volumeMA = input(title="volume ma period", type=input.integer, defval=200)
volMultiplyer = input(title="volume multiplyer", type=input.integer, defval=3)
donchianupper = input(title="upper donchian channel", type=input.integer, defval=1440)
donchianlower = input(title="lower donchian channel", type=input.integer, defval=1440)
takeprofit = input(title="take profit", type=input.float, defval=1000.00, step=0.01)
stops = input(title="stops", type=input.float, defval=500.00, step=0.01)

// V A R S
short = false
long = false
stopshort = close + stops
stoplong = close - stops

//----------------------?
//symbol = syminfo.ticker

// D O N C H I A N   C H A N N E L S
lower = lowest(donchianlower)
upper = highest(donchianupper)
middle= (upper-lower)/2+lower
plot(middle, title='Middle', color=color.green, linewidth=1)
plot(upper, title='Upper', color=color.orange, linewidth=1)
plot(lower, title='Lower', color=color.orange, linewidth=1)

// S M A
plot(sma(close,sma))

// V O L U M E
volMA = sma(volume, 200)
//upperVol = highest(volume, 200)
upperVol = volMA*volMultiplyer

// E N T R Y / E X I T   C o n d i t i o n s 
// S h o r t
if high>=upper and volume>upperVol
    short:=true
else 
    short:=false

// L o n g :
if low<=lower and volume>upperVol
    long:=true
else
    long:=false

// O R D E R S :
TP = takeprofit * 100
strategy.entry("short", strategy.short, limit=upper, stop=1, when=short)
strategy.entry("long", strategy.long, limit=lower, stop=1, when=long)
strategy.exit("tp-short", from_entry="short", profit=TP)
strategy.exit("tp-long", from_entry="long", profit=TP)