+global\ReleaseNotification::requireDefaultRecords()

Complexity

Conditions	9
Paths	10

Size

Total Lines	55
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function requireDefaultRecords()
    {
        parent::requireDefaultRecords();

        // verbose messages
        DB::alteration_message('Searching for environment configuration for ' . Director::absoluteURL('/'), 'created');

        // get the config based on the URL (URL is key)
        $config = $this->prepConfig($this->config()->get('environments'), Director::absoluteURL('/'));

It seems like \Director::absoluteURL('/') targeting Director::absoluteURL() can also be of type false; however, ReleaseNotification::prepConfig() does only seem to accept string, did you maybe forget to handle an error condition?

        // only run if everything is fine
        if (!is_array($config) || empty($config)) {
            DB::alteration_message('No configuration found.', 'created');
        } else if (!file_exists('../' . $config['filename'])) {
            DB::alteration_message('No CHANGELOG.md-file found.', 'created');
        } else {
            // get the information in the database for the last release notification
            $record = (self::get()->count() == 0) ? new self() : self::get()->first();

            // load the changelog
            $changelog = file_get_contents('../' . $config['filename']);

            // check if the CHANGELOG.md file has been changed since the last run
            if ($changelog != '' && md5($changelog) != md5($record->Changelog)) {
                // email the changelog out
                foreach ($config['recipients'] as $recipient) {
                    Email::create(
                        $config['from'],
                        $recipient,
                        $config['subject'],

                        // remove the old part and add more useful information to the content
                        sprintf(
                            "%s (%s)\n\n%s",
                            $config['environment_name'],
                            $config['url'],
                            $changelog
                        )
                    )->sendPlain();

                    DB::alteration_message($recipient . ' notified', 'created');
                }

                // save the new changelog to ensure we aren't re-running this in the next step
                $record->Changelog = $changelog;
            }

            // say welcome :)
            if (!$record->ID) {
                DB::alteration_message('Install of FriendsOfSilverStripe/release-notifications');
            }

            $record->write();
        }
    }