Bug 117703 - Word Count: Wrong result for "Characters excluding spaces"
Summary: Word Count: Wrong result for "Characters excluding spaces"
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.1.0.0.alpha1+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:7.1.0
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Word-Count FastParser-regressions
  Show dependency treegraph
 
Reported: 2018-05-19 08:06 UTC by raal
Modified: 2022-09-30 12:22 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments
test file (10.24 KB, application/vnd.oasis.opendocument.text)
2018-05-19 08:07 UTC, raal
Details
description -printscreens (64.43 KB, application/vnd.oasis.opendocument.text)
2018-05-19 08:07 UTC, raal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description raal 2018-05-19 08:06:20 UTC
Description:
New regression in 6.1

Steps to Reproduce:
1.Open file and select the characters from 24 to 39 position – words „a little bit of“
2.open Tools- Word Count
3.

Actual Results:  
Characters including spaces = 0. 

Expected Results:
Should be 44.


Reproducible: Always


User Profile Reset: No



Additional Info:


User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0
Comment 1 raal 2018-05-19 08:07:01 UTC
Created attachment 142201 [details]
test file
Comment 2 raal 2018-05-19 08:07:33 UTC
Created attachment 142202 [details]
description  -printscreens
Comment 3 raal 2018-05-19 08:46:11 UTC
This seems to have begun at the below commit.
Adding Cc: to Mohammed Abdul Azeem ; Could you possibly take a look at this one?
Thanks
 2fb92e5b348a9aa7fb86fe470d28e5fb8e743a27 is the first bad commit
commit 2fb92e5b348a9aa7fb86fe470d28e5fb8e743a27
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Wed Mar 14 14:28:34 2018 +0100

    source bf46b46a1d734348096936284fb8a76e977936d0
author	Mohammed Abdul Azeem <azeemmysore@gmail.com>	2017-10-01 14:11:11 +0530
committer	Michael Meeks <michael.meeks@collabora.com>	2018-03-14 14:12:08 +0100
commit bf46b46a1d734348096936284fb8a76e977936d0 (patch)
tree a76c30897cf1e49491bf26580c823701b6d74778
parent 1b61d0417bf46896ef1f1bd1e1a8209588fc157a (diff)
Moving XSAXDocumentBuilder2 to use XFastDocumentHandler:
Comment 4 Dieter 2018-05-19 11:49:14 UTC
I assume, you're talking about "characters excluding spaces" (see your attachment)

Repro with

Version: 6.1.0.0.alpha1+ (x64)
Build ID: 775d0f26beecffccf3ed27a6a011aff20d91f842
CPU threads: 4; OS: Windows 10.0; UI render: default; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2018-04-26_01:05:25
Locale: en-US (de_DE); Calc: CL
Comment 5 raal 2018-05-19 12:43:05 UTC
(In reply to Dieter Praas from comment #4)
> I assume, you're talking about "characters excluding spaces" 
Yes, thanks!
Comment 6 QA Administrators 2019-06-29 02:58:13 UTC Comment hidden (obsolete)
Comment 7 Timur 2020-09-01 09:18:32 UTC
Repro LO 6.3, no repro 7.0. WFM.
Comment 8 Timur 2020-09-01 09:41:49 UTC
LO 6.4:  bf884b45fb3fa38f127b490f61db77e7352f7be6 is the first good commit
Date:   Fri May 22 19:14:43 2020 +0200
    source d6c54d439e35e90dc51ddcbe2e0701ee22a5b9e1
Previous source 3ac066a585c781e0cb6e7fa7ae6a3c1107839dce
    
https://gerrit.libreoffice.org/plugins/gitiles/core/+/d6c54d439e35e90dc51ddcbe2e0701ee22a5b9e1%5E!/

commit d6c54d439e35e90dc51ddcbe2e0701ee22a5b9e1	[log]
author	himajin100000 <himajin100000@gmail.com>	Tue May 19 14:32:52 2020 +0200
committer	Michael Stahl <michael.stahl@cib.de>	Fri May 22 18:50:33 2020 +0200
tree d6502880363a30a6066897d6c5e3444efe133f39
parent 3ac066a585c781e0cb6e7fa7ae6a3c1107839dce [diff]

meta:non-whitespace-character-count attribute is known
also, fixed a typo and re-positioned the item to the correct index.
Comment 9 Xisco Faulí 2020-09-04 14:09:27 UTC
Verified in

Version: 7.1.0.0.alpha0+
Build ID: 6b2eff7d69c6e14d89dd33eaa58c01d82c541266
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

UItest submitted to gerrit
Comment 10 Commit Notification 2020-09-04 15:31:40 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/127696ba2b090c7ff93b6944b840e25acad63975

tdf#117703: sw: Add UItest

It will be available in 7.1.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.