[1.13] TextCacheProvider

Das Forum ist für das Melden von Bugs gedacht. // This forum is intended to report bugs with the APF.
Gesperrt
Benutzeravatar
Screeze
Beiträge: 1920
Registriert: 05.08.2009, 09:49:04
Kontaktdaten:

[1.13] TextCacheProvider

Beitrag von Screeze » 24.08.2010, 18:47:28

Code: Alles auswählen

public function clear(CacheKey $cacheKey = null){

         $baseFolder = $this->getConfigAttribute('Cache.BaseFolder');
         $namespace = str_replace('::','/',$this->getConfigAttribute('Cache.Namespace'));

         if($cacheKey === null){
            FilesystemManager::deleteFolder($baseFolder,true);
         }
         else{

            $cacheFile = $this->getCacheFile($cacheKey);
            try {
               FilesystemManager::removeFile($file);
               return true;
            } catch(FileException $e){
               return false;
            }
         }

       // end function
      }
>>FilesystemManager::removeFile($file);
$file ist nicht definiert, es müsste $cacheFile benutzt werden denke ich.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4536
Registriert: 04.11.2007, 16:13:53

Re: [1.13] TextCacheProvider

Beitrag von dr.e. » 25.08.2010, 09:58:10

Hallo Ralf,

korrekt, das ist falsch. Ich habe das im SVN gleich behoben.
Viele Grüße,
Christian

nxw
Beiträge: 16
Registriert: 01.10.2009, 17:24:11

[1.16] TextCacheProvider

Beitrag von nxw » 27.04.2013, 11:10:41

Der Thread ist zwar schon etwas älter, aber ich wollte dafür jetzt kein neues Thema aufmachen..

Ist es beabsichtigt, dass die clear-Methode bei leerem $cacheKey immer false zurückgibt?

Ich hätte jetzt die folgende Zeile:

Code: Alles auswählen

FilesystemManager::deleteFolder($baseFolder, true);
durch diese hier ersetzt:

Code: Alles auswählen

return FilesystemManager::deleteFolder($baseFolder, true);

Benutzeravatar
dr.e.
Administrator
Beiträge: 4536
Registriert: 04.11.2007, 16:13:53

Re: [1.13] TextCacheProvider

Beitrag von dr.e. » 28.04.2013, 12:40:44

Hallo nxw,

das ist ein guter Einwand. Ich korrigiere das im SVN gleich für 2.0.
Viele Grüße,
Christian

Gesperrt

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 1 Gast