This is not a bad effort. It gives you a good basic way to organize ads, but there are some flaws. First, if its main focus is on banner advertising, it should provide an upload interface internally. It took me a few minutes to realize I first had to upload an image to the banners folder in the media manager. I am also not sure why you have to specify the image size twice – you need to set it on the location management, then specifically again for the image when you create a new banner. Enable/Disable buttons on the banner list would be better if they were clickable. Also, in the module you have to manually enter the location id as a parameter, where this should be implemented as a drop-down list. Also, for the module it will not display the title I set even when I specifically enable it in the standard module options. The interface in general is very redundant which could confuse some users. I do like the ease of putting in the custom code, which is how I would be doing most of my adverts anyway. The restriction options based on content are nice, would be even better if you could specify components. There is a little extra tedious work to be done, but it gets the job done. Will look into why it is not displaying the title in the module.