Bug in the PluginController
Martin Häcker <[email protected]> Tue, 16 Aug 2005 16:41:35 +0200
| Newsgroups | gmane.network.fire.devel |
|---|---|
| Message-ID | <[email protected]> |
--Apple-Mail-9--132792734
Content-Transfer-Encoding: 7bit
Content-Type: text/plain;
charset=US-ASCII;
delsp=yes;
format=flowed
Hi there,
There was a small bug in the PreferencesController that left out the
incommingRawMessage case.
Also there was a naming inconsistency in the informal protocol which
I fixed.
Have Phun,
cu Martin
--Apple-Mail-9--132792734
Content-Transfer-Encoding: 7bit
Content-Type: application/octet-stream;
x-unix-mode=0644;
name="ctlToRaw.patch"
Content-Disposition: attachment;
filename=ctlToRaw.patch
Index: FireGrowlPlugin/FireGrowlPlugin.m
===================================================================
--- FireGrowlPlugin/FireGrowlPlugin.m (revision 4309)
+++ FireGrowlPlugin/FireGrowlPlugin.m (working copy)
@@ -287,7 +287,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
Index: FirePluginTemplate/TemplateClass.m
===================================================================
--- FirePluginTemplate/TemplateClass.m (revision 4309)
+++ FirePluginTemplate/TemplateClass.m (working copy)
@@ -211,7 +211,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
Index: FireSQLLogger/FireSQLLogger.m
===================================================================
--- FireSQLLogger/FireSQLLogger.m (revision 4309)
+++ FireSQLLogger/FireSQLLogger.m (working copy)
@@ -245,7 +245,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
Index: FireiAlertPlugin/FireiAlertPlugin.m
===================================================================
--- FireiAlertPlugin/FireiAlertPlugin.m (revision 4309)
+++ FireiAlertPlugin/FireiAlertPlugin.m (working copy)
@@ -242,7 +242,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
Index: FortuneProfilesPlugin/FortuneProfilesPlugin.m
===================================================================
--- FortuneProfilesPlugin/FortuneProfilesPlugin.m (revision 4309)
+++ FortuneProfilesPlugin/FortuneProfilesPlugin.m (working copy)
@@ -226,7 +226,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
Index: FireGrowlPlugin.10.2/FireGrowlPlugin.m
===================================================================
--- FireGrowlPlugin.10.2/FireGrowlPlugin.m (revision 4309)
+++ FireGrowlPlugin.10.2/FireGrowlPlugin.m (working copy)
@@ -275,7 +275,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
return NO;
}
--Apple-Mail-9--132792734
Content-Transfer-Encoding: 7bit
Content-Type: application/octet-stream;
x-unix-mode=0644;
name="protocolInconsistency.patch"
Content-Disposition: attachment;
filename=protocolInconsistency.patch
Index: Plugins/GPGEncryption/GPGEncryptionPlugin.m
===================================================================
--- Plugins/GPGEncryption/GPGEncryptionPlugin.m (revision 4309)
+++ Plugins/GPGEncryption/GPGEncryptionPlugin.m (working copy)
@@ -177,7 +177,7 @@
* NO will let the message continue processing
*/
-- (BOOL)incomingCtlMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
+- (BOOL)incomingRawMessage:(MessageItem *)msg additionalData:(NSDictionary *)dict
{
NSData *rawMessage = [msg messageData];
NSString *tStr = nil;
Index: Core/Controllers/PluginController.m
===================================================================
--- Core/Controllers/PluginController.m (revision 4309)
+++ Core/Controllers/PluginController.m (working copy)
@@ -374,11 +374,14 @@
SEL theSelector;
switch(type){
+ case FIRE_PLUGIN_OUTGOING_RAW:
+ theSelector = @selector(outgoingRawMessage:additionalData:);
+ break;
case FIRE_PLUGIN_OUTGOING_MSG:
theSelector = @selector(outgoingMessage:additionalData:);
break;
case FIRE_PLUGIN_INCOMING_RAW:
- theSelector = @selector(incomingCtlMessage:additionalData:);
+ theSelector = @selector(incomingRawMessage:additionalData:);
break;
case FIRE_PLUGIN_INCOMING_MSG:
theSelector = @selector(incomingMessage:additionalData:);
@@ -399,9 +402,6 @@
mustContinue = YES;
theSelector = @selector(quitting:additionalData:);
break;
- case FIRE_PLUGIN_OUTGOING_RAW:
- theSelector = @selector(outgoingRawMessage:additionalData:);
- break;
default:
return NO;
}
--Apple-Mail-9--132792734
Content-Transfer-Encoding: 7bit
Content-Type: text/plain;
charset=US-ASCII;
delsp=yes;
format=flowed
p.s.: I just noticed that in the last patch I forgot to update the
old project file - sorry.
--Apple-Mail-9--132792734--
-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf