Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

remove logging function (polluting unit tests) & bugfix on model subscriber addToSegments

Browse Source
This commit is contained in:
Jonathan Labreuille
2016-02-18 09:49:03 +01:00
parent 6a2e18a0e1
commit 0daf7e12c1
2 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/Config/Initializer.php
View File

@ -56,9 +56,6 @@ class Initializer {
\ORM::configure('username', Env::$db_username);
\ORM::configure('password', Env::$db_password);
\ORM::configure('logging', WP_DEBUG);
\ORM::configure('logger', function($query, $time) {
error_log("\n(Time: ".sprintf('%05f', $time).")".$query."\n");
});
\ORM::configure('driver_options', array(
\PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8',

13
lib/Models/Subscriber.php
View File

@ -35,10 +35,15 @@ class Subscriber extends Model {
function addToSegments(array $segment_ids = array()) {
$wp_users_segment = Segment::getWPUsers();
// delete all relations to segments except WP users
SubscriberSegment::where('subscriber_id', $this->id)
->whereNotEqual('segment_id', $wp_users_segment->id)
->deleteMany();
if($wp_users_segment !== false) {
// delete all relations to segments except WP users
SubscriberSegment::where('subscriber_id', $this->id)
->whereNotEqual('segment_id', $wp_users_segment->id)
->deleteMany();
} else {
// delete all relations to segments
SubscriberSegment::where('subscriber_id', $this->id)->deleteMany();
}
if(!empty($segment_ids)) {
$segments = Segment::whereIn('id', $segment_ids)->findMany();