Sorry about the missing include file. Reason I didn't include it is because that file calls a bunch of stuff in our custom backend and I cannot give you all the files to that.
Without being able to use the include I can hardly judge how much space your google ads eat up, so I can't really advise anything.
Thanks for taking the time to look into this, but why the angry undertone of the post?
Because you made several mistakes that could easily have been avoided by reading the docs we provide. Especially if you're a pro you should know better...
Board rules (you agreed to respect them when signing up) say that you're
only allowed to post here
after having read the docs. You obviously failed to do so. I provide support most eagerly if I can see that a user really has tried hard before asking. I couldn't see that level of effort from you.
Additionally, you picked a bad theme to customize the colors in the first place: you have to re-do all the graphics, changing the colors there. This resulted in not-so-nicely looking pixelation (you would have had to vectorize the graphics, then apply the changes using proper anti-aliasing, then convert back to bitmaps and finally chopping the resulting bitmap into tiny bits). In my opinion your customization took a lot of work, but the result doesn't look nice due to the pixelation.
Including the file /home/chefpla/public_html/admin/common.inc.php twice (in page header and page footer) is not bright neither, and the object oriented approach breaks the theme entirely if the include (which is just meant to serve some ads) goes down. This is not robust design. Google ads are just plain HTML and JavaScript anyway, so I can see little point in not adding just that into template.html instead of doing complicated stuff with includes.
Second, why do I have to rename the theme? I fail to see the relevance of that.
This is being
explained in the docs as well:
It is strongly recommended to rename your custom theme as suggested above, even if you only plan to accomplish minor changes to a default theme that coppermine comes with. The reason is quite simple: when upgrading at a later stage, you will not run into issues (e.g. accidentally overwrite your customized theme with an updated default theme).
This is particularly important as you say that you're applying the customization for a client - that person won't be aware of your changes to a core theme and will almost certainly accidentally overwrite the customization when applying the next maintenance release (and there is almost certainly a "next release", because development doesn't halt). See what I mean: you haven't read the docs...
![Roll Eyes ::)](https://forum.coppermine-gallery.net/Smileys/default/rolleyes.gif)
Do it now - it contains loads of important information.