Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

"Empty path is unavailable" - error after cancelling profile export #5146

Closed
tox-user opened this issue May 17, 2018 · 0 comments
Closed

"Empty path is unavailable" - error after cancelling profile export #5146

tox-user opened this issue May 17, 2018 · 0 comments
Labels
C-bug The issue contains a bug report

Comments

@tox-user
Copy link
Contributor

tox-user commented May 17, 2018

Brief Description

OS: Debian
qTox version: 1.15.0
Commit hash: 1f58e1c
toxcore: 0.2.2

Reproducible: Always

Steps to reproduce
  1. Go to profile settings.
  2. Press the "Export" button.
  3. Press cancel.
Observed Behavior

Error that says: "Empty path is unavailable".

Expected Behavior

Simply close the file manager without showing an error.

@tox-user tox-user changed the title Empty path is unavailable error after cancelling profile export "Empty path is unavailable" - error after cancelling profile export May 17, 2018
@tox-user tox-user added the C-bug The issue contains a bug report label May 17, 2018
ezavod added a commit to ezavod/qTox that referenced this issue May 29, 2018
Fixes qTox#5146. If the filedialog is cancelled the returned path is empty.
Do not try to export the profile in this case.
ezavod added a commit to ezavod/qTox that referenced this issue May 29, 2018
Fixes qTox#5146. If the filedialog is cancelled the returned path is empty.
Do not try to export the profile in this case.
HoraceBianchon added a commit to HoraceBianchon/qTox that referenced this issue Jul 17, 2018
* docs: Add a pull request template file

Add a pull request template to tell contributors that their commits
should follow the project's git commit guidelines before submitting
a pull request.

* chore(windows): bump OpenSSL version to the latest (1.0.2o)

* chore(i18n): translate INSTALL.md to persian

Translating INSTALL.md to Persian language.

* chore(windows): bump Qt 5.9 version to the latest (5.9.4)

* chore(windows): bump SQLCipher version to the latest (3.4.2)

* chore(windows): bump FFmpeg 3.2 version to the latest (3.2.10)

* chore(windows): bump QREncode version to the latest (4.0.0)

* chore(windows): bump Sodium version to the latest (1.0.16)

* chore(windows): bump VPX version to the latest (1.7.0)

* chore(windows): add architecture to installer's and zip's filenames

Since those are the filenames that will be used in the releases

* chore(windows): use NSIS 3 and cache NSIS ShellExecAsUser plugin

* chore(windows): create a zip with qTox to be used in release publishing

* fix(ui): remove placeholder update UI

Auto-updates were disabled in 196529b
Fix qTox#5040

* fix(settings): add missing enableTestSound to settings

* fix(chatform): include pressed key(s) when changing focus

Instead of just giving focus, give focus and add the key that was pressed. Also change from KeyRelease to KeyPress to avoid missing the second key pressed in the case of the second key being pressed before the first is released.

* fix(profile): don't error if no new avatar is selected

* feat(ui): Add ability to disable LAN discovery

Fix qTox#4074

* feat(ui): add setting for disabling new message notification popup

Fix qTox#4979

* refactor(ui): consolidate new message settings under UI page

* fix(audio): move audio timers to and start from audioThread

Fix qTox#5053

* fix(audio): apply gain to both audio channels

Fix qTox#5039

* refactor(avform): simplify and standardize sliders, use RMS for volume

Also improve usefulness of volume bar by including gain, clipping, and activation threshold. Remove magic numbers. Clear volumue display when mic is disabled.

Fix qTox#4893

* feat(camera): add higher resolution camera options

Fix qTox#5065

* fix(core): split messages on utf8 multibyte character boundary properly

Fix qTox#4917

* fix(settings): connect enable LAN discovery checkbox

Checkbox and setting introduced in 9f8b0fe but not connected.

* fix: delete double initialization callDuration

* perf(ui): cache stylesheets to reduce memory usage

Issue qTox#5078

* chore(version): update toxcore version to v0.2.2

* chore(windows): automatically deploy for windows

* chore(l10n): update translations from Weblate

Alessndro Magri (1):
      feat(l10n): update Italian translation from Weblate

Capsia Tech (3):
      feat(l10n): update Italian translation from Weblate
      feat(l10n): update Italian translation from Weblate
      feat(l10n): update Slovenian translation from Weblate

H Zeng (1):
      feat(l10n): update Chinese (Simplified) translation from Weblate

James Alison (58):
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate
      feat(l10n): update Persian translation from Weblate

Jan Kowalski (8):
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate

Nathan Follens (3):
      feat(l10n): add Flemish translation using Weblate
      feat(l10n): update Flemish translation from Weblate
      feat(l10n): update Flemish translation from Weblate

Oleh Ilnytskyi (2):
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Ukrainian translation from Weblate

Pan Efem (1):
      feat(l10n): update Polish translation from Weblate

TecHunter (2):
      feat(l10n): update French translation from Weblate
      feat(l10n): update French translation from Weblate

Unlucky Luke (1):
      feat(l10n): update Swedish translation from Weblate

Weblate (2):
      Merge remote-tracking branch 'origin/master'
      Merge remote-tracking branch 'origin/master'

anonymous (3):
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate
      feat(l10n): update Polish translation from Weblate

maclunar (1):
      feat(l10n): update Polish translation from Weblate

* chore(release): update qTox version to v1.15.0

* chore(windows): fix automatic deploy for windows

* fix(build): make qTox compile with ffmpeg 4.0 and newer

* fix(ui): increase number of low res camera options

partially revert behaviour of 04ecfe3 to show any options that were shown before. Mentioned in comments of qTox#5097.

* chore(release): Update README download links and MAINTAINING.md

* fix(theme): clear stylesheet cache on theme colour change

Fix qTox#5092

* chore(legal): update copyright date to 2018 for all source files

* feat(call): add call end sound

Fix qTox#4905

* fix(login): start login screen on profile select by -p option

* feat(appimage): build appimage on TravisCI

first step on the way to qTox#4874

* fix(build): correct install script nsis for win64

fix qTox#4627

* refactor(group): use displayedNameChanged signal

* refactor(group): store peer list in a more intelligent way

* fix: two crashes, uncovered by the persistent groupchat patch

* refactor(docs): Fix grammer and wording in the cross-compile guide

Rewrote the cross-compile guide to make it easier to understand.

* fix(appimage): build sqlcipher form source

Needed since Debian version is broken https://bugs.debian.org/850421

* fix(history): don't save both action prefix and displayed name

Displayed name is already added to action messages when added to ChatForm. Saving in history as well results in the display name being displayed twice when loaded from history.

* feat(history): load set number of messages from history

Fix qTox#3124
Fix qTox#3004

Instead of loading a set 7 days of history. Better performance when there are lots of messages, and better context when friends haven't talked in over a week.

Removed historyBaselineDate, introduced in deb8440 to fix duplicate messages, but duplicate messages were very likely fixed by qTox#4607.

Also refactored history loading.

* chore(windows): bump Qt 5.9 version to the latest (5.9.5)

* chore: typos

* chore(windows): the installer does get built now

* refactor(docs): re-format Windows cross-compilation instructions

* refactor(windows): remove the unused SCRIPT_DIR variable

* refactor(windows): remove the script directory requirement

* feat(chat): full screen video chat

Implements qTox#2922 and qTox#2514.
This change adds:
- a button for enabling full screen mode
- a panel with buttons for controlling the chat in full screen mode
- a button to toggle video preview
- new icons

fix(chat): fix buttons in full screen video call
feat(chat): add hotkey for exiting full screen video
fix(chat): use screen res to position button panel
fix(chat): dont remove video widget on window close

* refactor: use ToxPk as identifier

fixes qTox#5116 and qTox#5117

* chore(install): rename .desktop and .appdata.xml

The xdg-desktop specification strongly recommends to use a reverse
domain name, so follow that.

* feat: Add middle mouse clicked signal for GenericChatroom

* feat: Add ability to quit group with middle click

Fix qTox#2605

* feat: Add ability to remove dialog from content dialog with middle click

* fix(wayland): Fix desktop file name in Qt properties

fixes qTox#5141

* fix(core): Clean illegal chars from filenames

Fixes: qTox#1304

* chore: add more debugging when subscribing video

* fix(video): unsubscribe the video device correctly

fixes qTox#5110

* fix(audio): close the audio device after playing a sound

This bug was uncovered by 65896e4
because this sound now happens after the call is closed, but doesn't
close the audio device.

* fix(docs): update toxcore build instructions

Partially fixes qTox#5133

* chore(deploy): upload AppImage to Github releases on each tag

* fix(chatform): check for empty path when exporting profile

Fixes qTox#5146. If the filedialog is cancelled the returned path is empty.
Do not try to export the profile in this case.

* refactor(chatform): consistent usage of file dialogs

This makes the usage of QFileDialogs consistent by using he native
file picker in all cases. Also makes the usage of the last location
consistent. Furthermore this removes default parameter values and uses
Q_NULLPTR.

* refactor(chatform): Use QStringList instead of QVector<QString>

This replaces QVector<QString> with QStringList consistently across all files.

* feat(chat): add UI option to mute group peers

Implements qTox#4596, qTox#4626, qTox#1007

Adds a context menu to every peer label in group chat with an
option to mute chosen peers, which adds them to the blacklist.
Mutes audio of blacklisted group peers.

* fix(chatform): name in window title and close detached chats

This displays the name of the currently used account in the window title
of detached chats. Also closes all open chats when switching back to one
window.

* feat(core): put c-toxcore log messages in the qTox log

* fix(build): Elimination the build warnings (Wunused-variable, Wreorder)

* fix(leak): Fix few memory leaks

* fix: Fix usage of unitialized functions

* chore: Use '_FILENAME' to download archive

* fix(file): don't clean the filenames of avatar transfer

also don't print the filenames to the log for privacy reasons

* chore(windows): bump Qt 5.9 version to the latest (5.9.6)

* chore(windows): bump QREncode version to the latest (4.0.2)

* chore(windows): bump FFmpeg version to the latest (4.0.1)

* fix(Core): fix use after free of proxyAddrData

* fix: Fix PR qTox#5182. Eliminating the 'new' operator at ToxOptionsWrapper

* fix(settings): prevent segfault on wrong proxy settings

* docs(INSTALL): combine qtox and toxcore dependencies lists together

* chore(CI): add build for flatpak image

* fix: simple_make.sh script

- fix crash when run without parameters
- use cmake instad of automake for building toxcore

* docs(INSTALL): Add backslash to avoid error on install

* refactor(core): move logger to its own file

* refactor(core): move Tox_Options handling to separate file

* docs: update INSTALL.md

add instruction to install qtox in openSUSE

* refactor(toxcall): move alSource into ToxFriendCall

* refactor(toxcall): move peer handling to ToxGroupCall

* fix(audio): connect the correct audio callbacks

* fix: local toxcore install with bootstrap.sh

Fixes qTox#5199

* fix(simple_make): correct variable initialization

* fix: remove full screen btn from audio group chat

Fixes: qTox#5202

* chore(l10n): update translations from Weblate

Bogdan Khomutsky (1):
      feat(l10n): update Ukrainian translation from Weblate

Jonatan Nyberg (2):
      feat(l10n): update Swedish translation from Weblate
      feat(l10n): update Swedish translation from Weblate

Sonic Doom (1):
      feat(l10n): update Tamil translation from Weblate

Vasyan Nyasha (1):
      feat(l10n): update Ukrainian translation from Weblate

Weblate (1):
      Merge branch 'master' into weblate-merge-tmp

jji1902 (1):
      feat(l10n): update Korean translation from Weblate

Николай Спахиев (1):
      feat(l10n): update Bulgarian translation from Weblate

* fix(audio): fix error introduced in 67f2605

* fix(settings): automatically disable UDP when a proxy is set

fixes: qTox#5174

* chore: update dependencies to newest versions

* chore: update version number and changelog

* fix(deploy): fix file path in AppImage deployment

* feat(deploy): upload Flatpak bundle to Github releases

* chore(release): update version number and changelog

* refactor(core): cleanup Core class

- use a factory method to create it
- make it handle its own thread
- remove dependency on GUI

* refactor(core): fix error handling

* refactor(core): fix bootstrapping

* refactor(core): reimplement profile saving

* refactor(core): reimplement avatar transfers

In the process of this I used some ugly hacks, to finish this in time,
they should be changed as soon as possible.

* refactor(core): protect the tox main loop with a mutex

* refactor(core): use smart pointers instead of raw pointers

* refactor(core): remove unused return value and fix comment

* refactor(core): prepare for future error codes

* refactor(core): reimplement restart and small cleanups

* refactor(core): format code

* refactor(profile): small style corrections

* fix(profile): avoid deadlock with between main and core thread

The blocking connection was also blocking the callback holding the
coreLock.

* fix(chatform): mark message with triple click

Fixes qTox#5211. Only trigger on triple clicks that are caused by the same
mouse button clicked successively.

* docs: Update instructions to pull translations

* fix(logging): only log toxcore messages above TRACE level

* refactor: Add FriendChatroom (model to FriendWidget)

* chore(deploy): keep our own copy of the flatpak manifest in this repo

This allows us to easily add dependencies or change the build for new
features, but has the downside that we have to sync to the flathub.org
repo on a release

* fix(ui): don't emit filename change windows for every chat

* chore(README): add roadmap

* chore(docker): Add Dockerfile for debian and helper scripts

* chore(docker): Add Dockerfile for ubuntu and helper scripts

* docs(CONTRIBUTING):add a step within 'How to open a pull request'

* chore: add document about qTox TCS support state
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-bug The issue contains a bug report
Projects
None yet
Development

No branches or pull requests

1 participant