Currently it's string which kills some important use cases.

Files: 
CommentFileSizeAuthor
#2 0001-Fixed-1910126-Line-item-ID-Views-integration-should-.patch1.17 KBaxel.rutz
PASSED: [[SimpleTest]]: [MySQL] 3,570 pass(es).
[ View ]

Comments

Status:Active» Postponed (maintainer needs more info)

As an argument it is; I see that for the filter it's string, but is it also for the sort? What use cases are you talking about?

Status:Postponed (maintainer needs more info)» Reviewed & tested by the community
StatusFileSize
new1.17 KB
PASSED: [[SimpleTest]]: [MySQL] 3,570 pass(es).
[ View ]

Patch flying in.
It's trivial and working in production, so daring to rtbc (maintainer may feel free to bump back if disagree).

ah, ok - our posts had a race condition.

use cases / problems:
* filter "line item id <99" is not possible
* click sort is not numeric (1,15,2,...)

for me the filter thing was a real showstopper, the other a nice side effect.

(as of sort: sort happens in the db, so a sort handler is not typed. db knows its fields.)

Do you have the same issue then with the customer profile ID, order ID, and product ID, since their handlers extend views_handler_field instead of views_handler_numeric? Though looking at the core entity Views handlers, they also extend views_handler_field. Maybe it's not an issue for them?

Status:Reviewed & tested by the community» Fixed

In the absence of feedback, I'll commit this as is, since I haven't heard of problems with the other IDs. Also, I checked, and the payment transaction Views integration already used views_handler_field_numeric.

Commit: http://drupalcode.org/project/commerce.git/commitdiff/7ad17bc

> Do you have the same issue then with the customer profile ID, order ID, and product ID...?
order id really? can't believe i didn't bump into that.
if they are not numeric, i'll fix that as soon as i bump into it.

;-)

The filter is numeric, it's just the field handler itself that extends a different base field. But if it hasn't been an issue, I doubt it's an issue. As I said, we're following the same pattern for those entity ID fields as core uses.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.