getFile = QtWidgets.QFileDialog # Parenthesis are missing here

Your getFile object is not an instance of QFileDialog as you are missing the parenthesis. In this case, it's just an other name for QFileDialog.

I am not sure my esteemed colleague, @SGaist, looked closely enough at exactly what you have written :)

@SamuraiDestroy
You were not necessarily missing parentheses on getFile = QtWidgets.QFileDialog --- it depends which subsequent line you look at. It is correct for the getFile.getOpenFileName() statement but wrong for the getFile.selectedFiles() one.

You have a problem here with your attempted code. QtWidgets.QFileDialog.getOpenFileName() is a static method (see the docs) while QtWidgets.QFileDialog.selectedFiles() is an instance method (see the docs). This means you cannot use them together, they do not work on the same object. selectedFiles() will never look at or see what has happened in the getOpenFileName() call.

Are you wanting the user to open a single file or multiple files? If we assume single file then you do not need selectedFiles() at all and can use the simple static method:

(openedFile, filter) = QtWidgets.QFileDialog.getOpenFileName(None, "Open Image", "~", "Image Files (*.png *.jpg *.jpeg *.gif *.webp;;Project Files (*.ifx)")

The static getOpenFileName() puts up the dialog, waits for the user to select and press a button, and returns the (single) file selected (I think the empty string if the user presses Cancel, so you should check for that). Note that the the way the Python getOpenFileName() works, unlike the C++ one, is that it returns a two element list: the first element is the file selected and the second element is the filter chosen by the user, which you are rarely interested in. [Btw your Image files string clearly looks wrong --- count the parentheses --- so that may be interfering with its filter behaviour.]

If you really want more than that, such as the ability to return multiple files selected, you cannot use one of the static methods, you need to use an instance method to do the whole lot. Something like:

dialog = QtWidgets.QFileDialog()
dialog.setWindowTitle("Open Image")
dialog.setDirectory("~")    # if "~" works, else path to home directory
dialog.setFilter("Image Files (*.png *.jpg *.jpeg *.gif *.webp);;Project Files (*.ifx)")
dialog.setFileMode(QtWidgets.QFileDialog.ExistingFiles)
openedFiles = []
if dialog.exec():
    openedFiles = dialog.selectedFiles()

More complicated, and the sort of thing which the static getOpenFileName() does internally.

I suspect you do not want multiple files selected and need only the static code per the first example.

P.S.
I just noticed that even if you do want multiple files selected there is actually a static function for that too:

(openedFiles, filter) = QtWidgets.QFileDialog.getOpenFileNames(None, "Open Image", "~", "Image Files (*.png *.jpg *.jpeg *.gif *.webp);;Project Files (*.ifx)")

Notice getOpenFileNames() plural. So you could use that instead of the instance method. But I have left the instance stuff in, it can be useful in various circumstances where static methods do not do quite do what you want.

[Btw your Image files string clearly looks wrong --- count the parentheses --- so that may be interfering with its filter behaviour.]
Ah, I somehow missed that before. Yep, adding the extra ) fixed it.

@JonB said in TypeError: unbound method QFileDialog.selectedFiles() needs an argument:

I just noticed that even if you do want multiple files selected there is actually a static function for that too:

This works perfectly and does exactly what I was trying to do, thank you

@SGaist said in TypeError: unbound method QFileDialog.selectedFiles() needs an argument:

getFile = QtWidgets.QFileDialog # Parenthesis are missing here

Your getFile object is not an instance of QFileDialog as you are missing the parenthesis. In this case, it's just an other name for QFileDialog.

I am not sure my esteemed colleague, @SGaist, looked closely enough at exactly what you have written :)

I did but was way too short on my answer :-)
I somehow forgot to add the static function bit although I had it mind.

getFile = QtWidgets.QFileDialog # Parenthesis are missing here

Your getFile object is not an instance of QFileDialog as you are missing the parenthesis. In this case, it's just an other name for QFileDialog.

I am not sure my esteemed colleague, @SGaist, looked closely enough at exactly what you have written :)

@SamuraiDestroy
You were not necessarily missing parentheses on getFile = QtWidgets.QFileDialog --- it depends which subsequent line you look at. It is correct for the getFile.getOpenFileName() statement but wrong for the getFile.selectedFiles() one.

You have a problem here with your attempted code. QtWidgets.QFileDialog.getOpenFileName() is a static method (see the docs) while QtWidgets.QFileDialog.selectedFiles() is an instance method (see the docs). This means you cannot use them together, they do not work on the same object. selectedFiles() will never look at or see what has happened in the getOpenFileName() call.

Are you wanting the user to open a single file or multiple files? If we assume single file then you do not need selectedFiles() at all and can use the simple static method:

(openedFile, filter) = QtWidgets.QFileDialog.getOpenFileName(None, "Open Image", "~", "Image Files (*.png *.jpg *.jpeg *.gif *.webp;;Project Files (*.ifx)")

The static getOpenFileName() puts up the dialog, waits for the user to select and press a button, and returns the (single) file selected (I think the empty string if the user presses Cancel, so you should check for that). Note that the the way the Python getOpenFileName() works, unlike the C++ one, is that it returns a two element list: the first element is the file selected and the second element is the filter chosen by the user, which you are rarely interested in. [Btw your Image files string clearly looks wrong --- count the parentheses --- so that may be interfering with its filter behaviour.]

If you really want more than that, such as the ability to return multiple files selected, you cannot use one of the static methods, you need to use an instance method to do the whole lot. Something like:

dialog = QtWidgets.QFileDialog()
dialog.setWindowTitle("Open Image")
dialog.setDirectory("~")    # if "~" works, else path to home directory
dialog.setFilter("Image Files (*.png *.jpg *.jpeg *.gif *.webp);;Project Files (*.ifx)")
dialog.setFileMode(QtWidgets.QFileDialog.ExistingFiles)
openedFiles = []
if dialog.exec():
    openedFiles = dialog.selectedFiles()

More complicated, and the sort of thing which the static getOpenFileName() does internally.

I suspect you do not want multiple files selected and need only the static code per the first example.

P.S.
I just noticed that even if you do want multiple files selected there is actually a static function for that too:

(openedFiles, filter) = QtWidgets.QFileDialog.getOpenFileNames(None, "Open Image", "~", "Image Files (*.png *.jpg *.jpeg *.gif *.webp);;Project Files (*.ifx)")

Notice getOpenFileNames() plural. So you could use that instead of the instance method. But I have left the instance stuff in, it can be useful in various circumstances where static methods do not do quite do what you want.

    getFile = QtWidgets.QFileDialog # Parenthesis are missing here

Your getFile object is not an instance of QFileDialog as you are missing the parenthesis. In this case, it's just an other name for QFileDialog.