[Bug] pomo default break time is 25 min not 5 min??
Closed this issue · 1 comments
config = Pomo::Configuration.load(options.__hash__)
task = Pomo::Break.new('Break time', options.__hash__)
def self.load(options = {})
options.reject!{|k,v| ![:notifier, :progress, :tmux].include? k}
Because call reject! in the load method so the second options.__hash__
is {}, And it led to default break time to 25
I think save options.__hash__
to one variable and use it for Pomo::Configuration.load
and
Pomo::Break.new
may solve the problem.
I'm on vacation currently, but will look back into it when I get back home
letter this week.
On Tuesday, February 5, 2013, zlx wrote:
config = Pomo::Configuration.load(options.hash)task = Pomo::Break.new('Break time', options.hash)
def self.load(options = {})
options.reject!{|k,v| ![:notifier, :progress, :tmux].include? k}Because call reject! in the load method so the second options.hash is
{}, And it led to default break time to 25I think save options.hash to one variable and use it for
Pomo::Configuration.load and
Pomo::Break.new may solve the problem.—
Reply to this email directly or view it on GitHubhttps://github.com//issues/36.
-Steve