pt 0.7 → 0.7.1

data/Changelog.md

@@ -1,8 +1,13 @@
 # pt changelog
 
+## v0.7.1
+
+Fixes to pt show
+Updates to the help 
+Fixed the -m option 
+
 ## v0.7.0
 
-Fixed the -m option
 Comment, Assign, Estimate, Start, Finish, Deliver, Accept all accept the num value in `pt` or a story id
 
 ## v0.6.3 & v0.6.4

data/README.md

@@ -12,46 +12,48 @@ The first time you run it, `pt` will ask you some data about your Pivotal Tracke
 
 Run `pt` from the root folder of your project.
 
-    pt                                     # show all available tasks
+  pt                                         # show all available tasks
 
-    pt todo                                # show all unscheduled tasks
+  pt todo      <owner>                       # show all unscheduled tasks
 
-    pt started                             # show all started stories
+  pt started   <owner>                       # show all started stories
 
-    pt create    [title] ~[owner] ~[type]  # create a new task
+  pt create    [title] <owner> <type> -m     # create a new task (and include description ala git commit)
 
-    pt show      [id]                      # shows detailed info about a task
+  pt show      [id]                          # shows detailed info about a task
 
-    pt tasks     [id]                      # manage tasks of story
+  pt tasks     [id]                          # manage tasks of story
 
-    pt open      [id]                      # open a task in the browser
+  pt open      [id]                          # open a task in the browser
 
-    pt assign    [id] [member]             # assign owner
+  pt assign    [id] <owner>                  # assign owner
 
-    pt comment   [id] [comment]            # add a comment
+  pt comment   [id] [comment]                # add a comment
 
-    pt estimate  [id] [0-3]                # estimate a task in points scale
+  pt estimate  [id] [0-3]                    # estimate a task in points scale
 
-    pt start     [id]                      # mark a task as started
+  pt start     [id]                          # mark a task as started
 
-    pt finish    [id]                      # indicate you've finished a task
+  pt finish    [id]                          # indicate you've finished a task
 
-    pt deliver   [id]                      # indicate the task is delivered
+  pt deliver   [id]                          # indicate the task is delivered
 
-    pt accept    [id]                      # mark a task as accepted
+  pt accept    [id]                          # mark a task as accepted
 
-    pt reject    [id] [reason]             # mark a task as rejected, explaining why
+  pt reject    [id] [reason]                 # mark a task as rejected, explaining why
 
-    pt find      [query]                   # looks in your tasks by title and presents it
+  pt done      [id]  <0-3> <comment>         # lazy mans finish task, opens, assigns to you, estimates, finish & delivers
 
-    pt done      [id] ~[0-3] ~[comment]    # lazy mans finish task, does everything
+  pt find      [query]                       # looks in your tasks by title and presents it
 
-    pt list      [member]                  # list all tasks for another pt user
+  pt list      [owner]                       # list all tasks for another pt user
 
-  pt list      all                       # list all tasks for all users
+  pt list      all                           # list all tasks for all users
 
-    pt updates   [number]                  # shows number recent activity from your current project
+  pt updates   [number]                      # shows number recent activity from your current project
 
+  All commands can be run entirely without arguments for a wizard based UI. Otherwise [required] <optional>.
+  Anything that takes an id will also take the num (index) from the pt command.
 
 ## Problems?
 
@@ -73,4 +75,5 @@ You can [open a new issue](https://github.com/raul/pt/issues/new). It can be hel
 See the LICENSE file included in the distribution.
 
 ## Copyright
+Copyright (C) 2013 Orta Therox <orta.therox@gmail.com>.
 Copyright (C) 2011 Raul Murciano <raul@murciano.net>.

data/lib/pt/data_table.rb

@@ -15,6 +15,7 @@ module PT
         puts "\n#{'-- empty list --'.center(36)}\n"
       else
         self.class.table @rows, :fields => [:num] + self.class.fields,
+             :change_fields => %w{num pt_id},
              :unicode => true, :description => false,
              :max_width => config[:max_width]
       end

data/lib/pt/ui.rb

@@ -106,18 +106,17 @@ class PT::UI
 
     # did you do a -m so you can add a description?
     if ARGV.include? "-m" or ARGV.include? "--m"
-      description_file = Tempfile.new ["pt-create-description", "txt"]
-      description_file.write "Description for #{owner}'s #{task_type} ( delete this line )"
+      editor = ENV.fetch('EDITOR') { 'vi' }
+      temp_path = "/tmp/editor-#{ Process.pid }.txt"
+      system "#{ editor } #{ temp_path }"
       
-      system `#{ENV['EDITOR']} #{description_file.path}`
-      
-      description = File.read(description_file.path)
+      description = File.read(temp_path)
       result = @client.create_task_with_description(@project, name, owner, requester, task_type, description)
-
-      description_file.unlink      
+      
     else
       result = @client.create_task(@project, name, owner, requester, task_type)
     end
+    
     if result.errors.any?
       error(result.errors.errors)
     else
@@ -140,16 +139,6 @@ class PT::UI
     `open #{task.url}`
   end
 
-  def task_by_id_or_pt_id id
-    if id < 1000
-      tasks = @client.get_my_work(@project, @local_config[:user_name])
-      table = PT::TasksTable.new(tasks)
-      table[id]
-    else 
-      @client.get_task_by_id id
-    end
-  end
-
   def comment
     if @params[0]
       task = task_by_id_or_pt_id @params[0].to_i
@@ -347,9 +336,7 @@ class PT::UI
 
   def done
     if @params[0]
-      tasks = @client.get_my_work(@project, @local_config[:user_name])
-      table = PT::TasksTable.new(tasks)
-      task = table[@params[0].to_i]
+      task = task_by_id_or_pt_id @params[0].to_i
 
       #we need this for finding again later
       task_id = task.id
@@ -364,20 +351,20 @@ class PT::UI
             estimate_task(task, @params[1].to_i)
           end
           if @params[2]
-            task = find_my_task_by_task_id task_id
+            task = task_by_id_or_pt_id task_id
             @client.comment_task(@project, task, @params[2])
           end
       else
         @client.comment_task(@project, task, @params[1]) if @params[1]
       end
 
-      task = find_my_task_by_task_id task_id
+      task = task_by_id_or_pt_id task_id
       start_task task
 
-      task = find_my_task_by_task_id task_id
+      task = task_by_id_or_pt_id task_id
       finish_task task
 
-      task = find_my_task_by_task_id task_id
+      task = task_by_id_or_pt_id task_id
       deliver_task task
     end
   end
@@ -427,7 +414,7 @@ class PT::UI
 
   def find
     if (story_id = @params[0].to_i).nonzero?
-      if task = @client.get_task_by_id(story_id)
+      if task = task_by_id_or_pt_id(@params[0].to_i)
         return show_task(task)
       else
         message("Task not found by id (#{story_id}), falling back to text search")
@@ -467,13 +454,13 @@ class PT::UI
 
     title("Command line usage for pt #{PT::VERSION}")
     puts("pt                                         # show all available tasks")
-    puts("pt todo                                    # show all unscheduled tasks")
-    puts("pt started   ~[owner]                      # show all started stories")
-    puts("pt create    [title] ~[owner] ~[type] -m   # create a new task (and include description ala git commit)")
+    puts("pt todo      <owner>                       # show all unscheduled tasks")
+    puts("pt started   <owner>                       # show all started stories")
+    puts("pt create    [title] <owner> <type> -m     # create a new task (and include description ala git commit)")
     puts("pt show      [id]                          # shows detailed info about a task")
     puts("pt tasks     [id]                          # manage tasks of story")
     puts("pt open      [id]                          # open a task in the browser")
-    puts("pt assign    [id] ~[owner]                 # assign owner")
+    puts("pt assign    [id] <owner>                  # assign owner")
     puts("pt comment   [id] [comment]                # add a comment")
     puts("pt estimate  [id] [0-3]                    # estimate a task in points scale")
     puts("pt start     [id]                          # mark a task as started")
@@ -481,13 +468,13 @@ class PT::UI
     puts("pt deliver   [id]                          # indicate the task is delivered");
     puts("pt accept    [id]                          # mark a task as accepted")
     puts("pt reject    [id] [reason]                 # mark a task as rejected, explaining why")
+    puts("pt done      [id]  <0-3> <comment>         # lazy mans finish task, opens, assigns to you, estimates, finish & delivers")
     puts("pt find      [query]                       # looks in your tasks by title and presents it")
-    puts("pt done      [id] ~[0-3] ~[comment]        # lazy mans finish task, does everything")
-    puts("pt list      [owner]                       # list all tasks for another pt user")
-    puts("pt list      all                           # list all tasks for all users")
+    puts("pt list      [owner] or all                # list all tasks for another pt user")
     puts("pt updates   [number]                      # shows number recent activity from your current project")
     puts("")
-    puts("All commands can be run without arguments for a wizard like UI.")
+    puts("All commands can be run entirely without arguments for a wizard based UI. Otherwise [required] <optional>.")
+    puts("Anything that takes an id will also take the num (index) from the pt command.")
   end
 
   protected
@@ -620,6 +607,16 @@ class PT::UI
     nil
   end
 
+  def task_by_id_or_pt_id id
+    if id < 1000
+      tasks = @client.get_my_work(@project, @local_config[:user_name])
+      table = PT::TasksTable.new(tasks)
+      table[id]
+    else 
+      @client.get_task_by_id id
+    end
+  end
+
   def find_task query
     members = @client.get_members(@project)
     members.each do | member |
@@ -630,15 +627,6 @@ class PT::UI
     nil
   end
 
-  def find_my_task_by_task_id task_id
-      tasks = @client.get_my_work(@project, @local_config[:user_name])
-      tasks.each do |task|
-        if task.id == task_id
-          return task
-        end
-      end
-  end
-
   def find_owner query
     if query
       member = @client.get_member(@project, query)
@@ -684,15 +672,14 @@ class PT::UI
 
   def get_task_from_params(prompt)
     if @params[0]
-      task = @client.get_task_by_id(@params[0].to_i)
+      task = task_by_id_or_pt_id(@params[0].to_i)
     else
       tasks = @client.get_my_work(@project, @local_config[:user_name])
       table = PT::TasksTable.new(tasks)
       task = select(prompt, table)
     end
-    task
   end
-
+  
   def edit_story_task(story_task)
     action_class = Struct.new(:action, :key)
 

data/lib/pt.rb

@@ -2,7 +2,7 @@ require 'rubygems'
 
 module PT
   class InputError < StandardError; end
-  VERSION = '0.7'
+  VERSION = '0.7.1'
 end
 
 require 'pt/client'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pt
 version: !ruby/object:Gem::Version
-  version: '0.7'
+  version: 0.7.1
   prerelease: 
 platform: ruby
 authors:
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-08 00:00:00.000000000 Z
+date: 2013-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pivotal-tracker