6.0.0-beta1
7/20/25

[#12006] Quick Event Creation parser
Summary Quick Event Creation parser
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester delrio (at) mie (dot) utoronto (dot) ca
Created 01/30/2013 (4554 days ago)
Due
Updated 02/15/2013 (4538 days ago)
Assigned 02/11/2013 (4542 days ago)
Resolved 02/15/2013 (4538 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
02/15/2013 10:06:05 PM Git Commit Comment #12 Reply to this comment
Changes have been made in Git (master):

commit 1f2e172b3940b122e2dde7f73735fce925e97ce1
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Feb 15 17:05:27 2013 -0500

     Proper way to fix case preservation issues from Bug: 12006.

  .../lib/Horde/Date/Parser/Locale/Base.php          |    1 -
  .../lib/Horde/Date/Parser/Locale/Base/Repeater.php |   76 
++++++++++----------
  2 files changed, 38 insertions(+), 39 deletions(-)

http://git.horde.org/horde-git/-/commit/1f2e172b3940b122e2dde7f73735fce925e97ce1
02/15/2013 09:43:07 PM Michael Rubinsky Comment #11
State ⇒ Resolved
Reply to this comment
Tests fixed, all issues in ticket are fixed except the case not being 
preserved - which needs to be fixed in Horde_Support_Numerizer. I'll 
open an additional ticket for that.
02/12/2013 06:26:49 PM Michael Rubinsky Comment #10 Reply to this comment
This broke two unit tests.
*sigh* Was only running the parsing tests.

0dd572a831769a6f0e9e2ad32bd734f5cec33dbd and 
7c14c93d64a7d186a458e68fb81b3cd11d34823d fix one of the tests, will 
fix the other when I get more coding time.
02/11/2013 05:08:47 PM Jan Schneider Comment #9 Reply to this comment
Bump
02/11/2013 03:08:38 PM Jan Schneider Comment #8
Assigned to Michael Rubinsky
State ⇒ Assigned
Reply to this comment
This broke two unit tests.
02/01/2013 08:07:37 PM Michael Rubinsky Comment #7
Version ⇒ Git master
State ⇒ Resolved
Queue ⇒ Horde Framework Packages
Reply to this comment
Moving to Framework - problem was in Date_Parser.
02/01/2013 08:06:56 PM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (master):

commit 8f4d2d4ae24903b75610be382e7bf55ebdfd487c
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Feb 1 15:06:25 2013 -0500

     Preserve the case in non matched text.

     Bug: 12006

  .../lib/Horde/Date/Parser/Locale/Base.php          |   31 +++++++++----------
  .../test/Horde/Date/Parser/ParserTest.php          |   23 ++++++++++++++
  2 files changed, 38 insertions(+), 16 deletions(-)

http://git.horde.org/horde-git/-/commit/8f4d2d4ae24903b75610be382e7bf55ebdfd487c
02/01/2013 07:50:52 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit db96c8cf93d67241d99c446ae4412c135e6d1b0a
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Feb 1 14:49:16 2013 -0500

     Better parsing of timezone identifier.

     Bug: 12006

  .../lib/Horde/Date/Parser/Locale/Base/Timezone.php |    2 +-
  .../test/Horde/Date/Parser/ParserTest.php          |    1 +
  .../test/Horde/Date/Parser/TokenTest.php           |   26 
++++++++++++++++++++
  3 files changed, 28 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/db96c8cf93d67241d99c446ae4412c135e6d1b0a
02/01/2013 07:50:47 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit 2ebc8d77e140d048c608790078904201378bcefd
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Feb 1 13:53:30 2013 -0500

     Enforce timezone abbreviations occur at word boundry.

     Part of Bug: 12006

  .../lib/Horde/Date/Parser/Locale/Base/Timezone.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/2ebc8d77e140d048c608790078904201378bcefd
02/01/2013 07:50:40 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 0162d4419214ef4932297f884c3d53c4545c5a8f
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Feb 1 13:45:20 2013 -0500

     Ensure we have a numeric part when splitting off am or pm.

     Part of Bug: 12006

  .../lib/Horde/Date/Parser/Locale/Base.php          |    2 +-
  .../test/Horde/Date/Parser/ParserTest.php          |   16 ++++++++++++++++
  2 files changed, 17 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/0162d4419214ef4932297f884c3d53c4545c5a8f
01/30/2013 09:09:18 PM delrio (at) mie (dot) utoronto (dot) ca Comment #2 Reply to this comment
Some more testing:
Words ending in "am" are truncated, e.g.  "exam", "diagram"
Words containing "est" or any timezone letters make the parser fail, 
e.g. "testing" (EST), "upstate" (PST)

01/30/2013 06:14:27 PM delrio (at) mie (dot) utoronto (dot) ca Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Quick Event Creation parser
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
Adding events using Quick Event Creation feature in dynamic view:

"Lunch with David Sunday 11am"  event is added but description is all 
lowercase "lunch with david"
"Math exam Sunday 11am"  is added as "math ex"  (the word "exam" is truncated)
"Math test Sunday 11am" fails with "Cannot parse event description" 
(the word "test" seems to cause problems?)

Saved Queries