Skip to content

Commit fc3d341

Browse files
authored
Merge pull request #2996 from Mailaender/magic-chrom-converter
Check for magic byte and file content when converting chromatograms
2 parents 43d1bb0 + d18bf4b commit fc3d341

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

chemclipse/plugins/org.eclipse.chemclipse.converter/src/org/eclipse/chemclipse/converter/chromatogram/AbstractChromatogramConverter.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import org.eclipse.chemclipse.model.core.IPeak;
3232
import org.eclipse.chemclipse.model.support.ChromatogramColumnSupport;
3333
import org.eclipse.chemclipse.processing.DataCategory;
34+
import org.eclipse.chemclipse.processing.converter.ISupplier;
3435
import org.eclipse.chemclipse.processing.core.IProcessingInfo;
3536
import org.eclipse.chemclipse.processing.core.IProcessingMessage;
3637
import org.eclipse.chemclipse.processing.core.ProcessingInfo;
@@ -172,6 +173,14 @@ public IProcessingInfo getChromatogram(File file, boolean overview, IProgressMon
172173
List<String> availableConverterIds = converterSupport.getAvailableConverterIds(file);
173174
exitloop:
174175
for(String converterId : availableConverterIds) {
176+
try {
177+
ISupplier supplier = converterSupport.getSupplier(converterId);
178+
if(!supplier.isMatchMagicNumber(file) || !supplier.isMatchContent(file)) {
179+
continue;
180+
}
181+
} catch(NoConverterAvailableException e) {
182+
continue;
183+
}
175184
/*
176185
* Do not use a safe runnable here, because a IChromatogram
177186
* object must be returned or null.

0 commit comments

Comments
 (0)