View Issue Details

IDProjectCategoryView StatusLast Update
0003411Slicer4Core: Base Codepublic2018-05-30 12:48
ReporterjcfrAssigned Tojcfr 
PrioritynormalSeverityminorReproducibilityhave not tried
Status acknowledgedResolutionopen 
Product Version 
Target VersionFixed in Version 
Summary0003411: Externally set application home to CLIModuleFactoryHelper
Description

See https://github.com/Slicer/Slicer/blob/815612f613ab014707c7073b860e229ff6b06fde/Base/QTCLI/qSlicerCLIModuleFactoryHelper.cxx#L64-68

TagsNo tags attached.

Activities

lassoan

lassoan

2018-05-29 23:43

developer   ~0015787

Could you please describe what is the problem and what should be changed?

jcfr

jcfr

2018-05-30 12:48

administrator   ~0015845

The idea was to make the requirement for slicer home an explicit part of the contract by adding const QString& slicerHome as parameter to the different static methods.

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 4862d4ac

2013-09-21 22:44:52

jcfr

Details Diff
BUG: Fix crash in CLIModuleFactoryHelper when used without SlicerApp

The "isInstalled" method will alaways return false if no Slicer
application has been instantiated.

Ideally the application home should be externally set. This is
tracked by issue 0003411

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22464 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Base/QTCLI/qSlicerCLIModuleFactoryHelper.cxx Diff File

Issue History

Date Modified Username Field Change
2013-09-21 16:45 jcfr New Issue
2013-09-21 16:45 jcfr Status new => assigned
2013-09-21 16:45 jcfr Assigned To => jcfr
2013-09-21 16:45 jcfr Target Version => Slicer 4.4.0
2014-04-22 10:05 jcfr Status assigned => acknowledged
2014-04-22 10:05 jcfr Target Version Slicer 4.4.0 =>
2017-06-07 23:27 jcfr Changeset attached => Slicer 2145-support-for-installing-extension-from-file 4862d4ac
2018-05-29 23:43 lassoan Note Added: 0015787
2018-05-30 12:48 jcfr Note Added: 0015845