search

nytimes / Fech

Deprecated. Please see https://github.com/dwillis/Fech for a maintained fork.
http://nytimes.github.io/Fech/
Other
114 stars 30 forks source link

col_x_total_receipts_period returns value for col_x_total_disbursements #60

Closed capitolmuckrakr closed 10 years ago

capitolmuckrakr commented 10 years ago

It happens for col_a and col_b values.

dwillis commented 10 years ago

Thanks for this. A couple of questions: can you give me a filing where this happens? I can't find a col_b_total_receipts_period in the mappings - can you elaborate? Thanks.

dwillis commented 10 years ago

Sorry, found one (http://query.nictusa.com/cgi-bin/dcdev/forms/C00540617/890732/) and have replaced col_a_total_receipts_period with col_a_total_disbursements_period, but I can't find any reference to col_b_total_receipts_period. Once we clear that up, I can release a new version.

capitolmuckrakr commented 10 years ago

It's in http://query.nictusa.com/cgi-bin/dcdev/forms/C00258244/877551/. Seems as though they're redundant fields, since it's just repeating line 16, but who knows what the commission may decide tomorrow, plus easy enough to grab the wrong one.

dwillis commented 10 years ago

Right, got a fix for that, but you also mentioned col_b values, which I don't see in any mapping or filing.

capitolmuckrakr commented 10 years ago

Right, scratch what I just said. Now I can't find a col_b mapping either, although I thought I saw one earlier. I'll double check later, but I'm not at a terminal just now. In the meantime, let's assume I'm deluded and imagined it.

dwillis commented 10 years ago

570de59f0672ebddb50f2b940e20b5480138190d