----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109421/#review29217 -----------------------------------------------------------
This review has been submitted with commit c93f1e3f5602501f046a6874a6bf0bb33241ff11 by Sven Brauch to branch KDE/4.10. - Commit Hook On March 14, 2013, 3:24 p.m., Sven Brauch wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109421/ > ----------------------------------------------------------- > > (Updated March 14, 2013, 3:24 p.m.) > > > Review request for kdelibs and Jeremy Paul Whiting. > > > Description > ------- > > The download dialog correctly takes a custom providers .xml file, > as can be specified in the .knsrc file, into account; however the > upload dialog just ignored this option until now. This patch > intends to fix that behavior. > Nothing should change if you don't have a custom ProvidersUrl= in your > .knsrc file. > > > Diffs > ----- > > knewstuff/knewstuff3/upload/atticahelper.h 4e538d3 > knewstuff/knewstuff3/upload/atticahelper.cpp 735910f > knewstuff/knewstuff3/uploaddialog.cpp 70a8568 > knewstuff/knewstuff3/uploaddialog_p.h 50f8dd9 > > Diff: http://git.reviewboard.kde.org/r/109421/diff/ > > > Testing > ------- > > The new provider is correctly being listed in the dropdown list of the first > page of the dialog. If the custom provider is selected, the program tries to > establish a connection to that provider. However, I could not test it any > further because I don't have a working provider server... yet. > > > File Attachments > ---------------- > > a screenshot of the dialog showing two providers, one of them loaded from the > XML file specified in .knsrc > > http://git.reviewboard.kde.org/media/uploaded/files/2013/03/11/newdialog1.png > > > Thanks, > > Sven Brauch > >