pgbackman icon indicating copy to clipboard operation
pgbackman copied to clipboard

Add new method for output errors

Open rodo opened this issue 8 years ago • 3 comments

This will help to port to python3 and the print() method.

I do this first PR simple to open the discussion with you, are you agree with this kind of refacto ? I'll always try to not change any behaviour when doing refacto, so the output will be kept as they are actually

rodo avatar Nov 19 '16 09:11 rodo

It looks like a huge PR but I only add two functions, and always replace the same pattern in the whole file.

Replace

print '--------------------------------------------------------'
print '[ERROR]: ',e,'\n'

by

self.output_error(e)

rodo avatar Nov 19 '16 11:11 rodo

Ref #50

rodo avatar Nov 28 '16 08:11 rodo

Hello rodo

Thank you very much for your patch.

During the last week I have been testing a new release I have not had time to see this patch. The code is in freeze state until release. It is too late to have it in this release but I will merge to the repository in a few days.

regards

usit-gd avatar Nov 30 '16 15:11 usit-gd