sanoopsivan / jsmpp

Automatically exported from code.google.com/p/jsmpp
Apache License 2.0
0 stars 0 forks source link

Error parsing delivery reciept #6

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hello,

There is a typo in "org.jsmpp.bean.DeliveryReceipt" code.
You define DELREC_TEXT constant as "Text", but it should be "text" (at
least standard mentions it in lower case). Maybe it makes sense to search
for delivery receipt fields using case insensitive functions. 

Original issue reported on code.google.com by hudb...@gmail.com on 29 Aug 2008 at 12:07

GoogleCodeExporter commented 8 years ago
Hi, 

Thanks for your feedback. Can you tell me the standard you've been used. I'm 
using
SMPP version 3.4, it tells "Text" not "text". I don't know it's a typo or not, 
but
there is a jsmpp user who complained that it should be "Text" not "text" 
(because the
SMSC vendor use "Text" not "text"). I will appreciate if you give me the 
information
regarding to Delivery Receipt Format.

Thx

Original comment by uuda...@gmail.com on 31 Aug 2008 at 6:20

GoogleCodeExporter commented 8 years ago
Hello,

I'm using one from your site: SMPP_v3_4_Issue1_2, Appendix B, pages 167-168:
"text 20 Octet String The first 20 characters of the short message."
Two vendors I've checked use "text"..

Original comment by hudb...@gmail.com on 31 Aug 2008 at 11:43

GoogleCodeExporter commented 8 years ago
Well I believe we use the same one. I'm using "id:IIIIIIIIII sub:SSS dlvrd:DDD 
submit
date:YYMMDDhhmm done date:YYMMDDhhmm stat:DDDDDDD err:E Text:........." on the 
page
167 and you see on the Table B-1 on page 168. As we see on the table, the field 
name
is "text" not "Text". Well, I think I'll accommodate both variants, "text" is 
valid
and "Text" (with capital on the first letter) is valid too (since there are some
vendor using this). Thanks.

Original comment by uuda...@gmail.com on 31 Aug 2008 at 4:17