Unverified Commit 0851d3f7 authored by Marius van den Beek's avatar Marius van den Beek Committed by GitHub
Browse files

Merge pull request #19630 from mvdbeek/fail_sqlite_dataprovider

[24.2] Fail request explicitly when sqlite provider used on non-sqlite file
parents 7f71f2f1 4d5b8210
Loading
Loading
Loading
Loading
+6 −1
Original line number Diff line number Diff line
@@ -14,6 +14,7 @@ from bx import (
    wiggle as bx_wig,
)

from galaxy.exceptions import RequestParameterInvalidException
from galaxy.util import sqlite
from galaxy.util.compression_utils import get_fileobj
from . import (
@@ -716,7 +717,11 @@ class SQliteDataProvider(base.DataProvider):

    def __init__(self, source, query=None, **kwargs):
        self.query = query
        self.connection = sqlite.connect(source.dataset.get_file_name())
        datatype = source.datatype
        file_name = source.dataset.get_file_name()
        if not datatype.sniff(file_name):
            raise RequestParameterInvalidException("Dataset is not a SQlite file, cannot fetch data.")
        self.connection = sqlite.connect(file_name)
        super().__init__(source, **kwargs)

    def __iter__(self):