Opened 16 years ago

Closed 16 years ago

#67 closed defect (fixed)

Deprecate optional first argument to connect()

Reported by: [email protected] Owned by: alex
Priority: high Milestone:
Component: Core Version:
Keywords: Cc:
Blocked By: Blocking:


connect([type="after",] ...) is a confusing way to present an API. If you were learning by osmosis, you'd be very confused when you first encountered the uncommon cases of connect("before", ...) or connect("around", ...).

The easiest solution to this is to deprecate the optional-first-argument variant of connect, and simply use connectBefore and connectAround for the uncommon cases. It's shorter anyway, and far less confusing.

Explicit is better than implicit.

Change History (4)

comment:1 Changed 16 years ago by anonymous

Owner: changed from alex to anonymous
Status: newassigned

comment:2 Changed 16 years ago by alex

Owner: changed from anonymous to alex
Status: assignednew

comment:3 Changed 16 years ago by alex

Status: newassigned

comment:4 Changed 16 years ago by david

Resolution: fixed
Status: assignedclosed

(In [1138]) added connectAround and connectBefore shortcut functions, fixes #67

Note: See TracTickets for help on using tickets.