Problem dismissing UIImagePickerController camera overlay

[Blue Cup]

Hi all, first post here. I've been building apps for only a few weeks now but this is the first thing that's got me stumped!

I've added a custom overlay to a UIImagePickerController using the standard cameraOverlayView:

MainViewController.m

CameraOverlayView *overlay = [[CameraOverlayView alloc] initWithFrame:CGRectMake(0, 0, SCREEN_WIDTH, SCREEN_HEIGHT)];
picker.cameraOverlayView = overlay;

On the overlay I have a cancel button:

CameraOverlayView.m

UIButton *cancelButton = [UIButton buttonWithType:UIButtonTypeRoundedRect];
[cancelButton setTitle:@"Cancel" forState:UIControlStateNormal];
cancelButton.frame = CGRectMake(10, 10, 100, 40);
[cancelButton addTarget:self action:@selector(cancel:) forControlEvents:UIControlEventTouchUpInside]; 
[self addSubview:cancelButton];

My problem is that tapping the cancel button does nothing. I've coded it to dismiss the modal like other apps I've done

CameraOverlayView.m

- (IBAction) cancel:(id)sender {	
	[self.delegate cameraOverlayDidFinish:self];
}

which should run this:

MainViewController.m

- (void)cameraOverlayDidFinish:(CameraOverlayView *)cameraOverlayView
{ 
	[self dismissModalViewControllerAnimated:YES]; 
}

But nothing happens, no errors, nothing logged to the console. Not a thing.

I think I may need to be calling imagePickerControllerDidCancel like the standard picker uses, but if I change my cancelButton's selector to this it logs but then nothing either:

- (IBAction) cancel:(id)sender {
	NSLog(@"cancel tapped");
	[self.delegate imagePickerControllerDidCancel:(UIImagePickerController *)self];
}

Any help or suggestions would be very much appreciated, thanks!

[Blue Cup]

In case it helps, here's my complete code:

MainViewController.h

#import <UIKit/UIKit.h>
#import "CameraOverlayView.h"


@interface MainViewController : UIViewController <UINavigationControllerDelegate, UIImagePickerControllerDelegate, CameraOverlayDelegate> {

	IBOutlet UIImageView *imageView;
	IBOutlet UIBarButtonItem *takePhotoButton;
	IBOutlet UIButton *selectImageButton;

}

- (IBAction) getPictureFromCamera:(id)sender;
- (IBAction) getPictureFromLibrary;
- (void)cameraOverlayDidFinish:(CameraOverlayView *)cameraOverlayView;
- (void)imagePickerControllerDidCancel:(UIImagePickerController *)picker;

@property (nonatomic, retain) UIImageView *imageView;
@property (nonatomic, retain) UIBarButtonItem *takePhotoButton;

@end

MainViewController.m

#import "MainViewController.h"
#import "CameraOverlayView.h"

//transform values for full screen support
#define CAMERA_TRANSFORM_X 1
#define CAMERA_TRANSFORM_Y 1.12412

//iphone screen dimensions
#define SCREEN_WIDTH  320
#define SCREEN_HEIGHT 480

@implementation MainViewController

@synthesize imageView, takePhotoButton;


- (void)viewDidLoad {
    [super viewDidLoad];	
}

#pragma mark -
#pragma mark Camera picker

- (IBAction)getPictureFromCamera:(id)sender {
	//create an overlay view instance
	CameraOverlayView *overlay = [[CameraOverlayView alloc] initWithFrame:CGRectMake(0, 0, SCREEN_WIDTH, SCREEN_HEIGHT)];
	
	//create a new image picker instance
	UIImagePickerController *picker = [[UIImagePickerController alloc] init];
	picker.delegate = self;
	picker.sourceType = UIImagePickerControllerSourceTypeCamera;
	//hide all controls
	picker.showsCameraControls = NO;
	picker.navigationBarHidden = YES;
	picker.toolbarHidden = YES;
	//make the preview full size
	picker.wantsFullScreenLayout = YES;
	picker.cameraViewTransform = CGAffineTransformScale(picker.cameraViewTransform, CAMERA_TRANSFORM_X, CAMERA_TRANSFORM_Y);
	
	//set our custom overlay view
	picker.cameraOverlayView = overlay;
	
	//show picker
	[self presentModalViewController:picker animated:YES];

}

- (void)imagePickerController:(UIImagePickerController *)picker	didFinishPickingImage:(UIImage *)image editingInfo:(NSDictionary *)editingInfo {
	imageView.image = image;
	[picker dismissModalViewControllerAnimated:YES];
}

- (void)imagePickerControllerDidCancel:(UIImagePickerController *)picker {
	[picker dismissModalViewControllerAnimated:YES];
}

- (void)cameraOverlayDidFinish:(CameraOverlayView *)cameraOverlayView;
{ 
	[self dismissModalViewControllerAnimated:YES]; 
} 


#pragma mark -
#pragma mark Cleanup
- (void)didReceiveMemoryWarning {
    // Releases the view if it doesn't have a superview.
    [super didReceiveMemoryWarning];
    
    // Release any cached data, images, etc that aren't in use.
}

- (void)viewDidUnload {
	self.imageView = nil;
	self.takePhotoButton = nil;
    [super viewDidUnload];
}


- (void)dealloc {
	[imageView release];
	[takePhotoButton release];
    [super dealloc];
}


@end

CameraOverlayView.h

#import <UIKit/UIKit.h>

@protocol CameraOverlayDelegate;

@interface CameraOverlayView : UIView {
	id <CameraOverlayDelegate> delegate;
}

@property (nonatomic, assign) id <CameraOverlayDelegate> delegate;

@end

@protocol CameraOverlayDelegate <NSObject>
- (void)cameraOverlayDidFinish:(CameraOverlayView *)cameraOverlayView;
@end

CameraOverlayView.m

#import "CameraOverlayView.h"

@implementation CameraOverlayView

@synthesize delegate;

- (id)initWithFrame:(CGRect)frame {
    if (self = [super initWithFrame:frame]) {
        //clear the background color of the overlay
        self.opaque = NO;
        self.backgroundColor = [UIColor clearColor];
		
        //load an image to show in the overlay
        UIImage *searcher = [UIImage imageNamed:@"searcher.png"];
        UIImageView *searcherView = [[UIImageView alloc] initWithImage:searcher];
        searcherView.frame = CGRectMake(30, 100, 260, 200);
        [self addSubview:searcherView];
        [searcherView release];
		
		// cancel button...
        UIButton *cancelButton = [UIButton buttonWithType:UIButtonTypeRoundedRect];
        [cancelButton setTitle:@"Cancel" forState:UIControlStateNormal];
        cancelButton.frame = CGRectMake(10, 10, 100, 40);
		[cancelButton addTarget:self action:@selector(cancel:) forControlEvents:UIControlEventTouchUpInside]; 
        [self addSubview:cancelButton];
		
    }
    return self;
}

- (IBAction) cancel:(id)sender {	
	[self.delegate cameraOverlayDidFinish:self];
}

- (void)dealloc {
    [super dealloc];
}


@end

[Blue Cup]

Problem solved. I discovered Apple have a typically wonderful example app here: PhotoPicker

I've rewritten my code to work the same way.