dojo.connectPublisher simplification

[Les]

I believe the dojo.connectPublisher function can be simplified. This is because the event param is not optional according to the inline doc. If it is optional, then the doc should be corrected.

dojo.connectPublisher = function( /*String*/ topic, 
	/*Object|null*/ obj, 
	/*String*/ event){

	var pf = function(){ dojo.publish(topic, arguments); };
	return dojo.connect(obj, event, pf); //Handle
};

[dante]

so there isn't a test in place showing why one would ever use connectPublisher without an event. I tried to test this functionality, which I assume to be:

		function connectPublisherOptional(t){
			var obj = {
				val:0
			}
			function cb(){
				obj.val++;
			}
			var handle = dojo.connectPublisher("/hmm", cb) /* null event */
			dojo.subscribe("/hmm", function(){
				alert("was published.");
				obj.val++;
			});
			t.is(0, obj.val);
			cb();
			t.is(2, obj.val);
		},

though the subscription is never fired. I attempted also var cb = function(){} ... nothing seemed to publish on that topic.

[dante]

moving this to 1.6 ... 1.5 is expected soon, and changing this _may_ be an API change, and i'm simply missing the use case.

[bill]

dojo.connectPublisher() is in _base/connect.js, which implies that it's deprecated and will be removed in 2.0.