Skip to content
This repository was archived by the owner on Jul 1, 2026. It is now read-only.

Commit ee3d10a

Browse files
authored
Merge pull request #7 from akeneo/update_logger
Update logger to handle PIM fatal error responses
2 parents cee34ea + e3c967f commit ee3d10a

2 files changed

Lines changed: 24 additions & 3 deletions

File tree

.travis.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
language: php
2+
sudo: false
3+
php: 7.2
4+
5+
cache:
6+
directories:
7+
- $HOME/.composer/cache/files
8+
9+
install:
10+
- composer install
11+
12+
script:
13+
- ./vendor/bin/phpspec run

src/FileLogger.php

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,17 @@ public function logResponses(array $responses)
7272
$this->numUpdated++;
7373
break;
7474
default:
75-
$this->skip(
76-
sprintf('Skipped record "%s", an error occured during import: %s', $response['code'], json_encode($response['errors']))
77-
);
75+
if (isset($response['code']) && isset($response['errors'])) {
76+
$this->skip(
77+
sprintf(
78+
'Skipped record "%s", an error occured during import: %s',
79+
$response['code'],
80+
json_encode($response['errors'])
81+
)
82+
);
83+
} else {
84+
$this->skip(json_encode($response));
85+
}
7886
}
7987
}
8088
}

0 commit comments

Comments
 (0)