Added detection for Samsung C series printers #3598

Merged
merged 1 commit into from Jun 4, 2016

Projects

None yet

3 participants

@laf
Member
laf commented Jun 3, 2016

Fix #3584

@laf laf added the New-Device label Jun 3, 2016
@murrant murrant merged commit 29c8f55 into librenms:master Jun 4, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@paulgear paulgear commented on the diff Jun 8, 2016
includes/discovery/os/samsungprinter.inc.php
@@ -1,6 +1,8 @@
<?php
if (!$os) {
- if (strstr($sysDescr, 'Samsung CLX') || strstr($sysDescr, 'Samsung SCX')) {
- $os = 'samsungprinter';
+ if (strstr($sysDescr, 'Samsung CLX') ||
+ strstr($sysDescr, 'Samsung SCX') ||
+ strstr($sysDescr, 'Samsun C')) {
@paulgear
paulgear Jun 8, 2016 Member

Is this a typo for 'Samsung C', or do the printers actually report like that?

@laf laf deleted the laf:issue-3584 branch Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment