File Preprocessor Hook -file_object path property required?

Nils · July 19, 2023, 6:38am

Hi,

I am using the preprocessor hook to scan and change the GCODE file uploaded. Actually I am reading the temporally file uploaded (the original one) from file_object (octoprint.filemanager — OctoPrint master documentation) property "path" expecting DiskFileWrapper class. But it seems that some plugins adding files returning not the DiskFileWrapper but instead the AbstractFileWrapper or something else. There property "path" is missing which resulting into an exception with my plugin.

The reason I have to do a "scan" of the file is to set some common settings I need to do the line processing. The necessary information is not located at the top of every GCODE file.

github.com

NilsRo/OctoPrint-SlicerEstimator/blob/856e6c1ea8c65b4eed58ad22c3560f959cbdb32d/octoprint_SlicerEstimator/metadata.py#L114C1-L120C89


      
          class SlicerEstimatorFiledata(octoprint.filemanager.util.LineProcessorStream):
              def __init__(self, path, file_object, plugin):
                  super().__init__(file_object.stream())
                  self._logger = logging.getLogger("octoprint.plugins.SlicerEstimator")
                  self.path = path
                  self._file_object = file_object
                  self.slicer = SlicerEstimatorMetadataFiles.detect_slicer(self._file_object.path)

https://docs.octoprint.org/en/master/plugins/hooks.html#octoprint-filemanager-preprocessor

Does anybody knows what are the expected classes, is AbstractFileWrapper fine or this an issue caused by other plugins I ran into?

Cheers,
Nils