https://bugs.kde.org/show_bug.cgi?id=363176

--- Comment #7 from Johannes Huber <j...@gentoo.org> ---
(In reply to Johannes Huber from comment #5)
> ScriptConsole.cpp: 
> >   if( m_savePath.isEmpty() )
>  >      m_savePath = KUrl( KStandardDirs::locate( "data",
> "amarok/scriptconsole/" ) ).path();
> 
> According to kstandarddirs.h documentation: locateLocal should be used.

A second option would be to extend the 'if'  to check if dir is not writable:
   if( ( m_savePath.isEmpty() || !QDir( m_savePath ).exists() )
        && ( m_savePath = QFileDialog::getExistingDirectory(this, i18n( "Choose
where to save your scripts" ), "~",
            QFileDialog::ShowDirsOnly | QFileDialog::DontResolveSymlinks)
).isEmpty() )
        return 0;
 ts() )

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to