Net_RouterOS icon indicating copy to clipboard operation
Net_RouterOS copied to clipboard

Published 20 hours ago • verified publisher icon pear2

utils find implementation error

Open pavlyuts opened this issue 5 years ago • 4 comments
trafficstars

Hi!

There an arror in the code of find method, class util. As a result, it returns NOT comma separated, but just concatenated results, leading to any usage failure.

the path:

--- a/<html>Util.php (<b>Today 9:21:05</b>)</html>
+++ b/<html><b>Current File</b></html>
@@ -457,8 +457,8 @@
                     $idList .= strtolower(
                         is_string($newId)
                         ? $newId
-                        : stream_get_contents($newId) . ','
-                    );
+                        : stream_get_contents($newId)
+                    ) . ',';
                 }
             } elseif (is_callable($criteria)) {
                 $idCache = array();

pavlyuts avatar Nov 28 '19 06:11 pavlyuts

Last I checked, this is already resolved in dev-develop.

boenrobot avatar Nov 28 '19 09:11 boenrobot

Is it safe to use dev-develop in the producton?

pavlyuts avatar Nov 28 '19 10:11 pavlyuts

If you use a composer.lock file, sure.

I mean, new commits may come at any point, though at this point, I'm only doing minor fixes until I make an actual tagged release (which will happen when I can generate a new PHAR, which is a bit tricky...).

A composer.lock will ensure you stay at a particular commit that you've verified in a non-production environment.

boenrobot avatar Nov 28 '19 11:11 boenrobot

Thanks, clear!

pavlyuts avatar Nov 28 '19 11:11 pavlyuts