PKP Bugzilla – Full Text Bug Listing
|Summary:||Abstract word count limits appear to be incorrect|
|Product:||OCS||Reporter:||James MacGregor <jmacgreg>|
|Component:||General||Assignee:||PKP Support <pkp-support>|
|Severity:||normal||CC:||alec, jason.nugent, michael.pkp|
|Version Reported In:||Also Affects:|
Patch against OCS pre-2.3.4
Patch against OJS 2.4 stable
Description James MacGregor 2011-01-10 11:48:23 PST
Again, courtesy of http://pkp.sfu.ca/support/forum/viewtopic.php?f=3&t=7003&p=27045#p27045. It appears that abstract word counts are not correctly calculated, although while Trudy appears to be seeing a situation where more words than the limit can be added, I'm seeing one where drastically fewer words than the limit can be added (eg. when word limit is set to 200, I can only add 100). I can troubleshoot this further, but it would be nice to know what "counts" as a word.
Comment 1 Alec Smecher 2011-01-10 13:04:27 PST
Created attachment 3411 [details] Patch against OCS pre-2.3.4 The abstract word count didn't remove HTML tags before counting. The count will always be a little rough, since we're guessing the number of words in HTML content, but it should be closer than you report. Could you try the attached patch and see if it improves the situation?
Comment 2 James MacGregor 2011-08-15 13:20:44 PDT
(In reply to comment #1) > Created attachment 3411 [details] > Patch against OCS pre-2.3.4 > > The abstract word count didn't remove HTML tags before counting. The count will > always be a little rough, since we're guessing the number of words in HTML > content, but it should be closer than you report. Could you try the attached > patch and see if it improves the situation? Hi Alec -- that's much closer now, thanks!
Comment 3 Alec Smecher 2011-08-15 13:32:53 PDT
Thanks, James; committed against ocs-stable-2_3 and master.
Comment 4 Alec Smecher 2011-08-15 13:35:01 PDT
Comment 5 Michael Felczak 2014-05-23 05:15:10 PDT
This looks like it needs a port to ojs stable.
Comment 6 Jason Nugent 2014-05-23 05:52:03 PDT
Comment 7 Jason Nugent 2014-05-23 05:52:37 PDT
ported, closing again. Thanks for noticing, Michael.