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 #9927 (closed)

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

JsHint compliance

Reported by: wmoore Owned by: wmoore
Priority: major Milestone: OMERO-4.4.7
Component: Web Version: n.a.
Keywords: n.a. Cc: cxallan, cneves, jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: 2012-12-18 (3)

Description

We now have jsHint configured, but many files haven't been validated yet.

First target is webclient static js files.

One issue is global functions. These could be namespaced under 'OME' object, as I've started to do already for a couple of them.

Change History (4)

comment:1 Changed 11 years ago by wmoore

  • Status changed from new to accepted

comment:2 Changed 11 years ago by wmoore

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

comment:3 Changed 11 years ago by Will Moore <will@…>

(In [e445cc911151719e14a2c8034a2a3710dbdfbf04/ome.git] on branch develop) First round on jsHint validation fixes. See #9927

comment:4 Changed 11 years ago by Will Moore <will@…>

(In [66a186a422371307e6fc533fadffc8df6d54aed4/ome.git] on branch develop) Global functions in ome.popup.js moved to OME namespace object. See #9927

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.72567 sec.)

We're Hiring!