Codeception / Codeception/module-phalcon4
SessionManager methods missing
- Lingua principale
- PHP
- Stelle
- 3
- Fork
- 8
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
When my tests tried to run a test that involved a session destroy, I got an error. Because `MemorySession.php` doesn't actually start the session using `session_start`, `session_destroy` doesn't work either.
Also, `MemorySession.php`'s get and set methods aren't actually used, because the get and set are called from `SessionManager.php`.
I fixed this by adding methods to `SessionManager.php`:
```php
class SessionManager extends Manager
{
//...
public function destroy(): void
{
$this->getAdapter()->destroy($this->getId());
}
public function get(string $key, $defaultValue = null, bool $remove = false)
{
return $this->getAdapter()->get($key, $defaultValue, $remove);
}
public function set(string $key, $value): void
{
$this->getAdapter()->set($key, $value);
}
}
```
Please let me know if this is correct, or if I'm missing something.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia leggendo MemorySession.php e SessionManager.php, quindi riproduci il fallimento del test di distruzione della sessione descritto nell’issue. Controlla in che modo SessionManager delega le operazioni get, set e destroy al proprio adapter e verifica se MemorySession avvia le sessioni con session_start. Il lavoro è completato quando le operazioni della sessione funzionano tramite il manager e il test segnalato non fallisce più.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100