Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #2326 (closed)

Opened 14 years ago

Closed 14 years ago

Parameters auto-filled

Reported by: wmoore Owned by: wmoore
Priority: minor Milestone: OMERO-Beta4.2
Component: General Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: 2010-04-30 (8)

Description (last modified by wmoore)

If I define a parameter, E.g. scripts.Long("width", optional=True) but don't fill it in when running the script, I still find that after doing

for key in client.getInputKeys():
        if client.getInput(key):
            commandArgs[key] = client.getInput(key).getValue()

I find that commandArgs DOES have the "width" and it's value is 0L. This is causing me zero-division errors since I normally check whether the commandArgs has "width" and IF NOT, set it to sizeX.

Change History (5)

comment:1 Changed 14 years ago by wmoore

  • Description modified (diff)
  • Owner set to jmoore

comment:2 Changed 14 years ago by jmoore

  • Owner changed from jmoore to wmoore

Will, I'll be committing a passing test here shortly. I'm passing this back to you, because I can't reproduce. I was thinking about it: if you could attach a link to a failing test on hudson, that'd be optimal.

comment:3 Changed 14 years ago by jmoore

(In [6903]) see #2326 - Cannot reproduce autofill

comment:4 Changed 14 years ago by wmoore

  • Status changed from new to assigned

comment:5 Changed 14 years ago by wmoore

  • Resolution set to fixed
  • Status changed from assigned to closed

Seems to be working for me now.

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.71666 sec.)

We're Hiring!