Opened 14 years ago

Closed 14 years ago

#67 closed defect (fixed)

Deprecate optional first argument to connect()

Reported by: bob@… Owned by: alex
Priority: high Milestone:
Component: Core Version:
Keywords: Cc:
Blocked By: Blocking:

Description

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 14 years ago by anonymous

Owner: changed from alex to anonymous
Status: newassigned

comment:2 Changed 14 years ago by alex

Owner: changed from anonymous to alex
Status: assignednew

comment:3 Changed 14 years ago by alex

Status: newassigned

comment:4 Changed 14 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.