Custom action library, initial version

installer/src/installer-lib/record.cpp

Index: installer/src/installer-lib/record.cpp
===================================================================
new file mode 100644
--- /dev/null
+++ b/installer/src/installer-lib/record.cpp
@@ -0,0 +1,30 @@
+/**
+ * \file record.cpp Implementation of Session class.

[Wladimir Palant, 2013/10/29 08:49:26]

Nope, it's the Record class ;)

Generally, I'm unsure that this kind of abstraction is really necessary. Having
a class to abstract away one or two MSI function calls looks like an overkill to
me. We could use the same approach as with the Windows handles here - create an
AutoMSIHandle class that will hold an MSI handle and close it automatically in
the destructor. We can overload the cast to MSIHANDLE to make sure that helper
class can be used with any MSI functions directly.

[Eric, 2013/10/29 14:00:58]

Oops. Bad copypasta.
For the main reason this class exists, see the explanation on class Database.
Class Record is separate because it's used both in database queries and also as
an argument vector for MsiProcessMessage, which is currently used for logging
and we expect to use for dialog boxes in the CA proper. If we need offline
database tools, then we'll want this class. If we don't, then the right thing to
do is just to incorporate this whole class into a wrapper around
MsiProcessMessage. I don't know yet what we'll need.
That would be nice, if possible. The problem is that some handles are allocated
within the API as side effects, and you don't close those explicitly. Some (like
the one here for Record) do need to be explicitly closed. It's just not
consistent. On the other hand, I don't see a need to go full generic here and
use a trait class to manage the differences in handle lifetimes either; that's
premature at the very best.

I don't know what the case is with lifetimes for record handles generally.

[Wladimir Palant, 2013/10/29 15:06:38]

Well, these handles just don't need to be wrapped inside AutoMSIHandle then...

+ */
+
+#include "record.h"
+#include "msiquery.h"
+
+//-----------------------------------------------------------------------------------------
+// Record
+//-----------------------------------------------------------------------------------------
+Record::Record( unsigned int n_fields )
+  : n_fields( n_fields )
+{
+  _handle = MsiCreateRecord( n_fields ) ;
+  if ( ! _handle )
+  {
+    throw std::runtime_error( "Failed to create record" ) ;
+  }
+}
+
+Record::~Record()
+{
+  MsiCloseHandle( _handle ) ;
+}
+
+void 
+Record::assign_string( unsigned int field_index, std::wstring value )
+{
+  MsiRecordSetString( _handle, field_index, value.c_str() ) ;
+}