Refactor TemplateFileStream class using TemplateStream

Sming/Libraries/ArduCAM/ArduCAMStream.h

@@ -18,7 +18,7 @@ class ArduCAMStream: public ReadWriteStream {
 	ArduCAMStream(ArduCAM *cam);
 	virtual ~ArduCAMStream();
 
-	virtual StreamType getStreamType() { return eSST_User; }
+	virtual StreamType getStreamType() const { return eSST_User; }
 
 	virtual uint16_t readMemoryBlock(char* data, int bufSize);
 	virtual bool seek(int len);

Sming/SmingCore/Data/CircularBuffer.h

@@ -40,7 +40,7 @@ class CircularBuffer : public ReadWriteStream
      *  @retval StreamType The stream type.
      *  @todo   Return value of IDataSourceStream:getStreamType base class function should be of type StreamType, e.g. eSST_User
      */
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		return StreamType::eSST_Memory;
 	}
@@ -77,7 +77,7 @@ class CircularBuffer : public ReadWriteStream
 	 * @brief Returns unique id of the resource.
 	 * @retval String the unique id of the stream.
 	 */
-	virtual String id()
+	virtual String id() const
 	{
 		return String(reinterpret_cast<uint32_t>(&buffer), HEX);
 	}

Sming/SmingCore/Data/MailMessage.cpp

@@ -65,7 +65,7 @@ MailMessage& MailMessage::addAttachment(FileStream* stream)
 		return *this;
 	}
 
-	String filename = stream->fileName();
+	String filename = stream->getName();
 	String mime = ContentType::fromFullFileName(filename);
 
 	return addAttachment(stream, mime, filename);

Sming/SmingCore/Data/Stream/DataSourceStream.h

@@ -17,12 +17,13 @@
  *  @{
  */
 enum StreamType {
-	eSST_Memory,	   ///< Memory data stream
-	eSST_File,		   ///< File data stream
-	eSST_TemplateFile, ///< Template file data stream
-	eSST_JsonObject,   ///< JSON object data stream
-	eSST_User,		   ///< User defined data stream
-	eSST_Unknown	   ///< Unknown data stream type
+	eSST_Invalid,	///< Stream content not valid
+	eSST_Memory,	 ///< Memory data stream
+	eSST_File,		 ///< File data stream
+	eSST_Template,   ///< Template data stream
+	eSST_JsonObject, ///< JSON object data stream
+	eSST_User,		 ///< User defined data stream
+	eSST_Unknown	 ///< Unknown data stream type
 };
 /** @} */
 
@@ -43,7 +44,17 @@ class IDataSourceStream : public Stream
      *  @retval StreamType The stream type.
      *  @todo   Return value of IDataSourceStream:getStreamType base class function should be of type StreamType, e.g. eSST_User
      */
-	virtual StreamType getStreamType() = 0;
+	virtual StreamType getStreamType() const = 0;
+
+	/** @brief Determine if the stream object contains valid data
+	 *  @retval bool true if valid, false if invalid
+	 *  @note Where inherited classes are initialised by constructor this method indicates
+	 *  whether that was successful or not (e.g. FileStream)
+	 */
+	virtual bool isValid() const
+	{
+		return getStreamType() != eSST_Invalid;
+	}
 
 	/** @brief  Read a block of memory
      *  @param  data Pointer to the data to be read
@@ -61,7 +72,7 @@ class IDataSourceStream : public Stream
 
 	/**
 	 * @brief Read a character without advancing the stream pointer
-	 * @retval The character that was read or -1 if none is available
+	 * @retval int The character that was read or -1 if none is available
 	 */
 	virtual int peek();
 
@@ -116,7 +127,17 @@ class IDataSourceStream : public Stream
 	 * @brief Returns unique id of the resource.
 	 * @retval String the unique id of the stream.
 	 */
-	virtual String id()
+	virtual String id() const
+	{
+		return nullptr;
+	}
+
+	/**
+	 * @brief Returns name of the resource.
+	 * @retval String
+	 * @note Commonly used to obtain name of file
+	 */
+	virtual String getName() const
 	{
 		return nullptr;
 	}

Sming/SmingCore/Data/Stream/EndlessMemoryStream.h

@@ -30,7 +30,7 @@ class EndlessMemoryStream : public ReadWriteStream
 		delete stream;
 	}
 
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		return eSST_Memory;
 	}

Sming/SmingCore/Data/Stream/FileStream.cpp

@@ -91,19 +91,21 @@ bool FileStream::isFinished()
 	return fileIsEOF(handle);
 }
 
-String FileStream::fileName()
+String FileStream::fileName() const
 {
 	spiffs_stat stat;
-	fileStats(handle, &stat);
-	return String((char*)stat.name);
+	if(fileStats(handle, &stat) < 0)
+		return nullptr;
+	else
+		return String((char*)stat.name);
 }
 
-bool FileStream::fileExist()
+bool FileStream::fileExist() const
 {
 	return size != -1;
 }
 
-String FileStream::id()
+String FileStream::id() const
 {
 	spiffs_stat stat;
 	fileStats(handle, &stat);

Sming/SmingCore/Data/Stream/FileStream.h

@@ -30,7 +30,7 @@ class FileStream : public ReadWriteStream
 
 	virtual bool attach(const String& fileName, FileOpenFlags openFlags);
 	//Use base class documentation
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		return eSST_File;
 	}
@@ -47,8 +47,17 @@ class FileStream : public ReadWriteStream
 	//Use base class documentation
 	virtual bool isFinished();
 
-	String fileName(); ///< Filename of file stream is attached to
-	bool fileExist();  ///< True if file exists
+	String fileName() const; ///< Filename of file stream is attached to
+	bool fileExist() const;  ///< True if file exists
+	virtual String getName() const
+	{
+		return fileName();
+	}
+
+	virtual bool isValid() const
+	{
+		return fileExist();
+	}
 
 	/** @brief  Get the offset of cursor from beginning of data
      *  @retval int Cursor offset
@@ -67,7 +76,7 @@ class FileStream : public ReadWriteStream
 		return size;
 	}
 
-	virtual String id();
+	virtual String id() const;
 
 private:
 	file_t handle;

Sming/SmingCore/Data/Stream/JsonObjectStream.h

@@ -31,7 +31,7 @@ class JsonObjectStream : public MemoryDataStream
 	}
 
 	//Use base class documentation
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		return eSST_JsonObject;
 	}

Sming/SmingCore/Data/Stream/LimitedMemoryStream.cpp

@@ -18,7 +18,7 @@ LimitedMemoryStream::~LimitedMemoryStream()
 	delete[] buffer;
 }
 
-StreamType LimitedMemoryStream::getStreamType()
+StreamType LimitedMemoryStream::getStreamType() const
 {
 	return eSST_Memory;
 }

Sming/SmingCore/Data/Stream/LimitedMemoryStream.h

@@ -25,7 +25,7 @@ class LimitedMemoryStream : public ReadWriteStream
 	virtual ~LimitedMemoryStream();
 
 	//Use base class documentation
-	virtual StreamType getStreamType();
+	virtual StreamType getStreamType() const;
 
 	/**
 	 * @brief Return the total length of the stream

Sming/SmingCore/Data/Stream/MemoryDataStream.h

@@ -38,7 +38,7 @@ class MemoryDataStream : public ReadWriteStream
 	}
 
 	//Use base class documentation
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		return eSST_Memory;
 	}

Sming/SmingCore/Data/Stream/MultipartStream.h

@@ -36,7 +36,7 @@ class MultipartStream : public ReadWriteStream
 	virtual ~MultipartStream();
 
 	//Use base class documentation
-	virtual StreamType getStreamType()
+	virtual StreamType getStreamType() const
 	{
 		// TODO: fix this...
 		return stream->getStreamType();