-
Notifications
You must be signed in to change notification settings - Fork 29
Contribution-New Contact Form Plugin for Pawtucket #6
base: master
Are you sure you want to change the base?
Conversation
Can we make the "to email" configurable rather than hard-coding it in the code itself? |
Sure, making the "to email" configurable is a great idea; however since i'm new to CA i was not sure how to do this. I'm not sure why there were issues with the capitalization of file names but figured it had something to do with Zend naming conventions (i know Codeigniter can be picky about capitalization and how you name classes) but i don't use Zend so i was not sure if they had any rules. I'm running a localhost on my Mac and i kept getting "invalid controller path" errors until i de-capitalized front half of file names: formContactPlugin had to become formcontactPlugin and formContactController had to become formcontactController to get it to work. Then when i uploaded it our production server it would not work unless formcontactController became FormcontactController. I figured other people might run into the same type of issues, so that is why i left the note in there about possibly having to change the capitalization of file names. From: CollectiveAccess [email protected] Can we make the "to email" configurable rather than hard-coding it in the code itself? |
You can just put the email in the config file and pull it via the Configuration class. Macs are case insensitive (unless you specifically use a specific non-default file system), Linux is case-sensitive. I'll look at your code and see if I can find the problem. Thanks for contributing! |
Made requested changes. |
No description provided.