Overview
Comment:Actually make sure preCheck returns a bool
Downloads: Tarball | ZIP archive | SQL archive
Timelines: family | ancestors | descendants | both | dev
Files: files | file ages | folders
SHA1: 7d61ab1cfcfb2d174b0aa9f1a79093117b78437e
User & Date: bohwaz on 2020-12-04 19:25:09
Other Links: branch diff | manifest | tags
Context
2020-12-04
19:27
Make sure we don't get sent reminders linked to no reminder (weird) check-in: 4bd0ced0d8 user: bohwaz tags: dev
19:25
Actually make sure preCheck returns a bool check-in: 7d61ab1cfc user: bohwaz tags: dev
19:23
Make sure preCheck returns a boolean check-in: 89686bb4b8 user: bohwaz tags: dev
Changes

Modified src/include/lib/Garradin/Upgrade.php from [d98424ca2c] to [846a817db9].

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<?php

namespace Garradin;

use Garradin\Membres\Session;

class Upgrade
{
	static public function preCheck()
	{
		$config = Config::getInstance();
		$v = $config->getVersion();

		if (version_compare($v, garradin_version(), '>='))
		{
			throw new UserException("Pas de mise à jour à faire.");
		}

		if (!$v || version_compare($v, '0.9.8', '<'))
		{
			throw new UserException("Votre version de Garradin est trop ancienne pour être mise à jour. Mettez à jour vers Garradin 0.9.8 avant de faire la mise à jour vers cette version.");
		}









|






|







1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<?php

namespace Garradin;

use Garradin\Membres\Session;

class Upgrade
{
	static public function preCheck(): bool
	{
		$config = Config::getInstance();
		$v = $config->getVersion();

		if (version_compare($v, garradin_version(), '>='))
		{
			return false;
		}

		if (!$v || version_compare($v, '0.9.8', '<'))
		{
			throw new UserException("Votre version de Garradin est trop ancienne pour être mise à jour. Mettez à jour vers Garradin 0.9.8 avant de faire la mise à jour vers cette version.");
		}

31
32
33
34
35
36
37

38
39
40
41
42
43
44
				. PHP_EOL . $path);
		}

		// Voir si l'utilisateur est loggé, on le fait ici pour le cas où
		// il y aurait déjà eu des entêtes envoyés au navigateur plus bas
		$session = new Session;
		$user_is_logged = $session->isLogged(true);

	}

	static public function upgrade()
	{
		$config = Config::getInstance();
		$v = $config->getVersion();








>







31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
				. PHP_EOL . $path);
		}

		// Voir si l'utilisateur est loggé, on le fait ici pour le cas où
		// il y aurait déjà eu des entêtes envoyés au navigateur plus bas
		$session = new Session;
		$user_is_logged = $session->isLogged(true);
		return true;
	}

	static public function upgrade()
	{
		$config = Config::getInstance();
		$v = $config->getVersion();