Patch for /src/SeleniumLibrary/screenshot.py about oldseleniumlibrary HOT 4 CLOSED

Originally submitted to Google Code by jollych... on 25 Jul 2013

anybody can merge it?thanks

from oldseleniumlibrary.

Originally submitted to Google Code by @jussimalinen on 13 Aug 2015

We are moving Selenium Library to github and creating a last release with Robot 2.9 support before stopping the support of this project entirely.

We will look at merging the patch before creating the last release.

from oldseleniumlibrary.

Attachment cannot be copied from Google Code so here's the proposed patch:

--- a/src/SeleniumLibrary/screenshot.py
+++ b/src/SeleniumLibrary/screenshot.py
@@ -80,6 +80,8 @@
             filename = self._namegen.next()
         else:
             filename = filename.replace('/', os.sep)
+        import os
+        filename = filename.replace('.png', '%s.png' % os.urandom(16).encode('hex'))
         logdir = self._get_log_dir()
         path = os.path.join(logdir, filename)
         link = utils.get_link_path(path, logdir)

from oldseleniumlibrary.

We decided not to do the proposed change because it would change behavior in backwards incompatible manner. It could be considered in a major version, but because this library is deprecated we are not planning to release new major versions.

from oldseleniumlibrary.