diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2018-02-03 15:23:48 +0100 |
|---|---|---|
| committer | Katarina Behrens <Katarina.Behrens@cib.de> | 2018-02-06 11:36:01 +0100 |
| commit | 2e6bbc04d906ed308fc5ca769bad7b8b4676594e (patch) | |
| tree | 101a54e6d99cb5558d6eeb2dca7643f7982dfd73 /cui/source/options | |
| parent | b6db65752c60de7268e4f65d7fc5ac110adb2fb0 (diff) | |
tdf#114731: persona html can contain " instead of "
So let's replace all " with " in the retrieved html
Change-Id: I4a90ecc51e7ce6d1e080e6c1b17b82fe9a29d65e
Reviewed-on: https://gerrit.libreoffice.org/49179
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins <ci@libreoffice.org>
Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
Diffstat (limited to 'cui/source/options')
| -rw-r--r-- | cui/source/options/personalization.cxx | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/cui/source/options/personalization.cxx b/cui/source/options/personalization.cxx index c75a78c8ef2e..9b3e482b55d2 100644 --- a/cui/source/options/personalization.cxx +++ b/cui/source/options/personalization.cxx @@ -553,10 +553,13 @@ static OUString searchValue( const OString &rBuffer, sal_Int32 from, const OStri } /// Parse the Persona web page, and find where to get the bitmaps + the color values. -static bool parsePersonaInfo( const OString &rBuffer, OUString *pHeaderURL, OUString *pFooterURL, +static bool parsePersonaInfo( const OString &rBufferArg, OUString *pHeaderURL, OUString *pFooterURL, OUString *pTextColor, OUString *pAccentColor, OUString *pPreviewURL, OUString *pName ) { + // tdf#115417: buffer retrieved from html response can contain " or " + // let's replace the whole buffer with last one so we can treat it easily + OString rBuffer = rBufferArg.replaceAll(OString("""), OString(""")); // it is the first attribute that contains "persona=" sal_Int32 persona = rBuffer.indexOf( "data-browsertheme=\"{" ); if ( persona < 0 ) |