Variable fetched from wrong module name

Gah, I have such twisted fingers..

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>
Getting package name directly from the params class is bad

First it gets us an empty name in some situations. Second, it doesn't
take into account the value that the user would have passed to the main
class to override the name.

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>
Require and notify links are wrong

Currently, it's impossible to install asterisk because require and
notify links refer directly to package and service resouces.

First case: the main class already forces an order so inside install,
config and service classes those links are unnecessary.

Second case: for deeper-nested defines, we should require and notify the
install and the service classes instead of referencing things that are
done inside them. It simplifies links and ensures that we always refer
to the right thing even if things change inside those classes.

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>
Simplify iax and sip templates.

The comments that come with the default config in debian are great, but
they are useless in the context of a file managed by puppet.

Plus, they force us to special-case every darned option.

Let's stay simple and KISS all that cruft goodbye.

Note: the options from the hash are sorted so that the contents of the
files will not change on every puppet run.

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>
Collapse all configuration into asterisk::config

All of the classes that do only one thing, calling config_dotd, are
actually just configuration that should be done for the rest of the
module to work OK.

So let's put all of that stuff inside the config class and get rid of
those other classes. It'll centralize basic configuration and clean up
the module by reducing the number of files.

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>