Changeset 1562:26ae03082033
- Timestamp:
- 10/20/08 12:44:33
(3 months ago)
- Author:
- Dan Blankenberg <dan@bx.psu.edu>
- branch:
- default
- Message:
Fix for implicit conversion checking on multiple paged tools. Conversion is never allowed on multi-paged tools.
-
Files:
-
Legend:
- Unmodified
- Added
- Removed
- Modified
- Copied
- Moved
| r1557 |
r1562 |
|
| 1007 | 1007 | ToolParameter.__init__( self, tool, elem ) |
|---|
| 1008 | 1008 | # Add metadata validator |
|---|
| 1009 | | self.validators.append( validation.MetadataValidator() ) |
|---|
| | 1009 | if not str_bool( elem.get( 'no_validation', False ) ): |
|---|
| | 1010 | self.validators.append( validation.MetadataValidator() ) |
|---|
| 1010 | 1011 | # Build tuple of classes for supported data formats |
|---|
| 1011 | 1012 | formats = [] |
|---|
| … | … | |
| 1108 | 1109 | def dataset_collector( datasets ): |
|---|
| 1109 | 1110 | def is_convertable( dataset ): |
|---|
| 1110 | | target_ext, converted_dataset = dataset.find_conversion_destination( self.formats, converter_safe = True ) #need to assume converter_safe = True, since we don't know about other parameter values here |
|---|
| | 1111 | target_ext, converted_dataset = dataset.find_conversion_destination( self.formats, converter_safe = self.converter_safe( None, trans ) ) |
|---|
| 1111 | 1112 | if target_ext is not None: |
|---|
| 1112 | 1113 | return True |
|---|
| … | … | |
| 1181 | 1182 | |
|---|
| 1182 | 1183 | def converter_safe( self, other_values, trans ): |
|---|
| 1183 | | if not hasattr( trans, 'workflow_building_mode' ) or trans.workflow_building_mode: |
|---|
| | 1184 | if self.tool.has_multiple_pages or not hasattr( trans, 'workflow_building_mode' ) or trans.workflow_building_mode: |
|---|
| 1184 | 1185 | return False |
|---|
| | 1186 | if other_values is None: |
|---|
| | 1187 | return True # we don't know other values, so we can't check, assume ok |
|---|
| 1185 | 1188 | converter_safe = [True] |
|---|
| 1186 | 1189 | def visitor( prefix, input, value, parent = None ): |
|---|