Overview
Comment:Don't select preferences when sending emails, it's useless
Downloads: Tarball | ZIP archive | SQL archive
Timelines: family | ancestors | descendants | both | dev
Files: files | file ages | folders
SHA3-256: 659389412b1a11a87a2647cd0fed214590c8e1820de191a3d4cb7d09d9c94ef1
User & Date: bohwaz on 2023-04-28 20:09:41
Other Links: branch diff | manifest | tags
Context
2023-04-28
20:29
Refactor mailing creation, preview and sending check-in: 9b5769ef8f user: bohwaz tags: dev
20:09
Don't select preferences when sending emails, it's useless check-in: 659389412b user: bohwaz tags: dev
20:09
Make sure the spans also have an underline when linked check-in: 887461cde6 user: bohwaz tags: dev
Changes

Modified src/include/lib/Garradin/Users/Users.php from [71d7256705] to [3f4e070ed1].

38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
	{
		$db = DB::getInstance();
		$fields = DynamicFields::getEmailFields();
		$sql = [];
		$where = $id_category ? sprintf('id_category = %d', $id_category) : 'id_category IN (SELECT id FROM users_categories WHERE hidden = 0)';

		foreach ($fields as $field) {
			$sql[] = sprintf('SELECT *, %s AS _email FROM users WHERE %s AND %1$s IS NOT NULL', $db->quoteIdentifier($field), $where);
		}

		return $db->iterate(implode(' UNION ALL ', $sql));
	}

	/**
	 * Return a list of all emails by service (user must be active)







|







38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
	{
		$db = DB::getInstance();
		$fields = DynamicFields::getEmailFields();
		$sql = [];
		$where = $id_category ? sprintf('id_category = %d', $id_category) : 'id_category IN (SELECT id FROM users_categories WHERE hidden = 0)';

		foreach ($fields as $field) {
			$sql[] = sprintf('SELECT *, %s AS _email, NULL as preferences FROM users WHERE %s AND %1$s IS NOT NULL', $db->quoteIdentifier($field), $where);
		}

		return $db->iterate(implode(' UNION ALL ', $sql));
	}

	/**
	 * Return a list of all emails by service (user must be active)