silverstripe / silverstripe-dms

tests/Stub/StubRelatedDocumentExtension.php

<?php

/**
 * Class StubRelatedDocumentExtension
 *
 * @package dms
 */
class StubRelatedDocumentExtension extends DataExtension implements TestOnly

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * For method {@link DMSDocument::getRelatedDocuments}
     *
     * @param  ArrayList $relatedDocuments
     * @return ArrayList
     */
    public function updateRelatedDocuments($relatedDocuments)
    {
        $relatedDocuments->push($this->getFakeDocument());
        return $relatedDocuments;
    }

    /**
     * For method {@link DMSDocumentSet::getDocuments}
     *
     * @param  ArrayList $relatedDocuments

There is no parameter named $relatedDocuments. Did you maybe mean $documents?

     * @return ArrayList
     */
    public function updateDocuments($documents)
    {
        $documents->push($this->getFakeDocument());
        return $documents;
    }

    /**
     * Return a dummy document for testing purposes
     *
     * @return DMSDocument
     */
    protected function getFakeDocument()
    {
        $fakeDocument = new DMSDocument;
        $fakeDocument->Filename = 'Extended';
        return $fakeDocument;
    }
}