officecfg/registry/data/org/openoffice/Office/PresentationMinimizer.xcu | 3 + officecfg/registry/schema/org/openoffice/Office/PresentationMinimizer.xcs | 4 + sdext/source/minimizer/optimizerdialog.cxx | 29 +++------- sdext/source/minimizer/pppoptimizertoken.cxx | 2 sdext/source/minimizer/pppoptimizertoken.hxx | 1 5 files changed, 21 insertions(+), 18 deletions(-)
New commits: commit 25403bf60e9b3554f2cae118fe77c7075fb5bbaa Author: Vasily Melenchuk <vasily.melenc...@cib.de> AuthorDate: Fri Nov 2 13:53:16 2018 +0100 Commit: Thorsten Behrens <thorsten.behr...@cib.de> CommitDate: Tue Nov 6 15:48:12 2018 +0100 presentation minimizer: fixed problem with default filename According to code, it is expected that minimized copy of presentation is proposed with ".mini" suffix as default, but this does not works right now. As a solution a new localized prefix "(minimized)" is added to filename. Change-Id: If869c70a5885a9c917fc7254aa8b0d164a6f5a50 Reviewed-on: https://gerrit.libreoffice.org/62781 Tested-by: Jenkins Reviewed-by: Thorsten Behrens <thorsten.behr...@cib.de> Reviewed-on: https://gerrit.libreoffice.org/62964 Tested-by: Thorsten Behrens <thorsten.behr...@cib.de> diff --git a/officecfg/registry/data/org/openoffice/Office/PresentationMinimizer.xcu b/officecfg/registry/data/org/openoffice/Office/PresentationMinimizer.xcu index fab5d1945cc7..c13aeac41955 100644 --- a/officecfg/registry/data/org/openoffice/Office/PresentationMinimizer.xcu +++ b/officecfg/registry/data/org/openoffice/Office/PresentationMinimizer.xcu @@ -196,6 +196,9 @@ The current presentation contains no OLE objects.</value> <prop oor:name="STR_FILESIZESEPARATOR"> <value xml:lang="en-US">.</value> </prop> + <prop oor:name="STR_FILENAME_SUFFIX"> + <value xml:lang="en-US">(minimized)</value> + </prop> </node> <node oor:name="LastUsedSettings"> <prop oor:name="Name"> diff --git a/officecfg/registry/schema/org/openoffice/Office/PresentationMinimizer.xcs b/officecfg/registry/schema/org/openoffice/Office/PresentationMinimizer.xcs index 179637d7d677..446b0159a4b1 100644 --- a/officecfg/registry/schema/org/openoffice/Office/PresentationMinimizer.xcs +++ b/officecfg/registry/schema/org/openoffice/Office/PresentationMinimizer.xcs @@ -361,6 +361,10 @@ The current presentation contains no OLE objects.</value> <info><desc>String STR_FILESIZESEPARATOR.</desc></info> <value>.</value> </prop> + <prop oor:name="STR_FILENAME_SUFFIX" oor:type="xs:string" oor:localized="true"> + <info><desc>String STR_FILENAME_SUFFIX.</desc></info> + <value>(minimized)</value> + </prop> </group> <node-ref oor:name="LastUsedSettings" oor:node-type="OptimizerSettings"> <info> diff --git a/sdext/source/minimizer/optimizerdialog.cxx b/sdext/source/minimizer/optimizerdialog.cxx index ff8c6cb10b49..4b789ca7e408 100644 --- a/sdext/source/minimizer/optimizerdialog.cxx +++ b/sdext/source/minimizer/optimizerdialog.cxx @@ -513,24 +513,17 @@ void ActionListener::actionPerformed( const ActionEvent& rEvent ) Reference< XStorable > xStorable( mrOptimizerDialog.mxController->getModel(), UNO_QUERY ); if ( xStorable.is() && xStorable->hasLocation() ) { - OUString aLocation( xStorable->getLocation() ); - if ( !aLocation.isEmpty() ) - { - sal_Int32 nIndex = aLocation.lastIndexOf( '/', aLocation.getLength() - 1 ); - if ( nIndex >= 0 ) - { - if ( nIndex < aLocation.getLength() - 1 ) - aLocation = aLocation.copy( nIndex + 1 ); - - // remove extension - nIndex = aLocation.lastIndexOf( '.', aLocation.getLength() - 1 ); - if ( nIndex >= 0 ) - aLocation = aLocation.copy( 0, nIndex ); - - // adding .mini - aLocation = aLocation.concat( ".mini" ); - aFileOpenDialog.setDefaultName( aLocation ); - } + INetURLObject aURLObj( xStorable->getLocation() ); + if ( !aURLObj.hasFinalSlash() ) { + // tdf#105382 uri-decode file name + aURLObj.removeExtension(INetURLObject::LAST_SEGMENT, false); + auto aName( aURLObj.getName( INetURLObject::LAST_SEGMENT, + false, + INetURLObject::DECODE_WITH_CHARSET ) ); + // Add "(minimized)" + aName += " "; + aName += mrOptimizerDialog.getString(STR_FILENAME_SUFFIX); + aFileOpenDialog.setDefaultName( aName ); } } bool bDialogExecuted = aFileOpenDialog.execute() == dialogs::ExecutableDialogResults::OK; diff --git a/sdext/source/minimizer/pppoptimizertoken.cxx b/sdext/source/minimizer/pppoptimizertoken.cxx index d99c49bfb15b..e48bb05dfdba 100644 --- a/sdext/source/minimizer/pppoptimizertoken.cxx +++ b/sdext/source/minimizer/pppoptimizertoken.cxx @@ -152,6 +152,8 @@ static const TokenTable pTokenTableArray[] = { "STR_OPTIMIZING_GRAPHICS", STR_OPTIMIZING_GRAPHICS }, { "STR_CREATING_OLE_REPLACEMENTS",STR_CREATING_OLE_REPLACEMENTS }, { "STR_FileSizeSeparator", STR_FILESIZESEPARATOR }, + { "STR_FILENAME_SUFFIX", STR_FILENAME_SUFFIX }, + { "NotFound", TK_NotFound } }; diff --git a/sdext/source/minimizer/pppoptimizertoken.hxx b/sdext/source/minimizer/pppoptimizertoken.hxx index e458da0d3181..a72afd3ef83d 100644 --- a/sdext/source/minimizer/pppoptimizertoken.hxx +++ b/sdext/source/minimizer/pppoptimizertoken.hxx @@ -135,6 +135,7 @@ enum PPPOptimizerTokenEnum STR_OPTIMIZING_GRAPHICS, STR_CREATING_OLE_REPLACEMENTS, STR_FILESIZESEPARATOR, + STR_FILENAME_SUFFIX, TK_NotFound }; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits