Let CreatePartitionDialog create the Partition object

src/modules/partition/CreatePartitionDialog.cpp

@@ -64,5 +64,14 @@ CreatePartitionDialog::createJob()
 
     FileSystem::Type type = FileSystem::typeForName( m_ui->fsComboBox->currentText() );
     FileSystem* fs = FileSystemFactory::create( type, first, last );
-    return new CreatePartitionJob( m_device, m_freePartition, fs );
+
+    PartitionNode* parent = m_freePartition->parent();
+    Partition* partition = new Partition(
+        parent,
+        *m_device,
+        PartitionRole( PartitionRole::Primary ), // FIXME: Support extended partitions
+        fs, first, last,
+        QString() /* path */
+    );
+    return new CreatePartitionJob( m_device, partition );
 }

src/modules/partition/CreatePartitionJob.cpp

@@ -24,10 +24,9 @@
 #include <core/partitiontable.h>
 #include <fs/filesystem.h>
 
-CreatePartitionJob::CreatePartitionJob( Device* device, Partition* freePartition, FileSystem* fs )
+CreatePartitionJob::CreatePartitionJob( Device* device, Partition* partition )
     : m_device( device )
-    , m_freePartition( freePartition )
-    , m_fs( fs )
+    , m_partition( partition )
 {
 }
 
@@ -43,18 +42,9 @@ CreatePartitionJob::exec()
 }
 
 void
-CreatePartitionJob::createPreview()
+CreatePartitionJob::updatePreview()
 {
-    PartitionNode* parent = m_freePartition->parent();
-    Partition* partition = new Partition(
-        parent,
-        *m_device,
-        PartitionRole( PartitionRole::Primary ), // FIXME: Support extended partitions
-        m_fs, m_fs->firstSector(), m_fs->lastSector(),
-        QString() /* path */
-    );
-
     m_device->partitionTable()->removeUnallocated();
-    parent->insert( partition );
+    m_partition->parent()->insert( m_partition );
     m_device->partitionTable()->updateUnallocated( *m_device );
 }

src/modules/partition/CreatePartitionJob.h

@@ -28,11 +28,11 @@ class FileSystem;
 class CreatePartitionJob : public Calamares::Job
 {
 public:
-    CreatePartitionJob( Device* device, Partition* freePartition, FileSystem* fs );
+    CreatePartitionJob( Device* device, Partition* partition );
     QString prettyName() override;
     void exec() override;
 
-    void createPreview();
+    void updatePreview();
     Device* device() const
     {
         return m_device;
@@ -40,7 +40,7 @@ public:
 
 private:
     Device* m_device;
-    Partition* m_freePartition;
+    Partition* m_partition;
     FileSystem* m_fs;
 };
 

src/modules/partition/PartitionCoreModule.cpp

@@ -93,7 +93,7 @@ PartitionCoreModule::createPartition( CreatePartitionJob* job )
 {
     DeviceInfo* info = deviceInfoForDevice( job->device() );
     Q_ASSERT( info );
-    job->createPreview();
+    job->updatePreview();
     info->partitionModel->reload();
     Calamares::JobQueue::instance()->enqueue( Calamares::job_ptr( job ) );
 }