Audio File Processor

[cbj4074]

In the interest of ensuring that this since-deleted logic isn't overlooked when we write tests in relation to audio file processing, I'm hereby citing the relevant business rules:

https://github.com/indiehd/audio-manipulator/blob/7fc5d3ff29d46b2e5f3fe9b987f08614b1b6fe88/src/Flac/FlacConverter.php#L77-L113

and

https://github.com/indiehd/audio-manipulator/blob/7fc5d3ff29d46b2e5f3fe9b987f08614b1b6fe88/src/Flac/FlacConverter.php#L139-L160

// These toggles default to true, but may be changed hereafter, depending
// on the totality of the inputs supplied.

$performTrim = true;
$canFadeIn = true;
$canFadeOut = true;

// Determine whether or not the audio file is actually shorter than
// the specified clip length. This is done to prevent the resultant
// file from being padded with silence.

if ($fileDetails['playtime_seconds'] < $clipLength) {
    // No trim or fades are necessary if the play-time
    // is less than the specified clip length.

    $performTrim = false;
    $canFadeIn = false;
    $canFadeOut = false;
}

// This block prevents problems from a track preview start
// time that is too far into the track to allow for a preview clip of
// the specified length.

if (($fileDetails['playtime_seconds'] - $trimStartTime) < $clipLength) {
    // We'll force the track preview to start exactly $clipLength seconds
    // before the end of the track, thus forcing a $clipLength-second preview
    // clip length.

    $trimStartTime = $fileDetails['playtime_seconds'] - $clipLength;
}

// Convert the clip length from seconds to hh:mm:ss format.

$clipLength = Utility::sec2hms($clipLength, false, false);

if ($performTrim !== false) {
    $cmd[] = ' trim ' . escapeshellarg($trimStartTime) . ' ' . escapeshellarg($clipLength);
}

if ($canFadeIn !== false || $canFadeOut !== false) {
    $cmd[] = ' fade q ';

    if ($canFadeIn === false) {
        // Setting a fade-in length of zero in SoX is the
        // same as having no fade-in at all.

        $fadeInLength = '0';
    }

    $cmd[] = escapeshellarg($fadeInLength);

    if ($canFadeOut !== false) {
        $cmd[] = ' ' . escapeshellarg($clipLength) . ' ' . escapeshellarg($fadeOutLength);
    }
}

[cbj4074]

Similarly, there is valuable, application-specific logic here that will need to be preserved in some capacity:

https://github.com/indiehd/audio-manipulator/blob/7fc5d3ff29d46b2e5f3fe9b987f08614b1b6fe88/src/Flac/FlacTagger.php#L326-L414

[cbj4074]

I just released v2.0.0 of the required library, which is the version that should be required in this project, e.g., ^2.0.

[poppabear8883]

Will you be requiring that package directly in or are you bridging it with a Laravel package ?

[cbj4074]

That's a good point/question.

Some type of Laravel-specific bridge package would probably be ideal. I'll have to dig into it a bit deeper and see what the integration might look like. I'll post an update on this soon.