seed icon indicating copy to clipboard operation
seed copied to clipboard

Enable filtering of columns that appear in both inventory types

Open anchapin opened this issue 2 years ago • 2 comments

For example, the "Address Line 1" column for Properties can't be filtered in the Property version of Inventory List.

anchapin avatar Sep 01 '22 20:09 anchapin

https://github.com/SEED-platform/seed/pull/3515

Ryoken avatar Sep 12 '22 19:09 Ryoken

Issue #3426 is also about cross Inventory Filtering / Sorting.

RDmitchell avatar Oct 20 '22 16:10 RDmitchell

instance: staging SHA: bb2fdd425 Org: LBNL Test 200

Was able to filter on an Tax Lot field in the Property List tab.

However, when I tried to add a label to those filtered records, I got several 500 errors image

RDmitchell avatar Dec 06 '22 01:12 RDmitchell

the filter worked using text, ie, filtering on Parcel Owner == Lucky University image

However, it didn't seem to work when I filtered on the Jurisdiction Tax Lot ID for the case where there were multiple tax lot IDs associated with a record image But it doesn't seem to work even if there is only one Tax Lot ID associated with a record image But it did work filtering with a number on the address field image

RDmitchell avatar Dec 06 '22 01:12 RDmitchell

https://github.com/SEED-platform/seed/pull/3658 shoulf fix that. Thanks for the find.

haneslinger avatar Dec 06 '22 15:12 haneslinger

Instance: Dev1 SHA: 294626cbd Org: LBNL Test 200

Cycle: 2020 Compliance Cycle

Filtering for blank or not blank values in a Tax Lot field -- Does not work In this example, there are 14 records in the cycle. 2 of them do not have a value in the Jurisdiction Tax Lot ID column image

If I filter for the records with the Tax Lot field blank, ie, enter "" in the filter field, no records are returned. But the two records with that field blank should be returned image

This same style of filter, ie, "" to show blanks, works for Property fields image


Cycle: 2018 Compliance Cycle

Here is another case, that is equally broken but somehow in a different way

Total number of records == 19 image

Address Line 1 (Tax Lot) field

Using the Legacy Property List

  • using "" as the filter for blank records == 7 records that have this field blank image

  • using !"" as the filter for not blank records == 12 records where that field is not blank image

Using the New Property List

  • using "" as the filter for blank records == 1 record, but that field isn't actually blank for that record image

  • using !"" as the filter for not blank records == 17 records where that field is not blank, except it should be 12. There appear to be duplicates of some of the records, and if you de-duplicate them, I believe you are left with 12 records. image

RDmitchell avatar Jan 12 '23 01:01 RDmitchell

Instance: Dev1 SHA: https://github.com/SEED-platform/seed/commit/294626cbdbdd7c89dd14f2f85f17e633ed8e4367 Org: LBNL Test 200 Cycle: 2018 Compliance Cycle

Can't filter on ; in Tax Lot field in Property Tab in the Legacy Property List, if you filter on ; for Jurisdiction Tax Lot ID, it returns the 5 records that have a semicolon image

In the new Property List, doing that same filter results in 0 records image

RDmitchell avatar Jan 12 '23 01:01 RDmitchell

Filtering on Tax Lot columns that appear in the Property Tab still needs a lot of work.

I am putting this back in the "In Progress" column, as it is not completely fixed.

RDmitchell avatar Jan 12 '23 02:01 RDmitchell

Filtering on "" was broken for all columns. Reported here. Fixed here. Not yet released. Moving this to "Ready to deploy"

haneslinger avatar Jan 12 '23 20:01 haneslinger

@haneslinger -- great, thanks !!

RDmitchell avatar Jan 12 '23 20:01 RDmitchell

@haneslinger -- what about filtering on semicolon, ;

That also seems broken

RDmitchell avatar Jan 12 '23 20:01 RDmitchell

and possibly other punctuation?

RDmitchell avatar Jan 13 '23 00:01 RDmitchell

Instance: dev1 SHA: 26aac34d8 Org: LBNL Test 200 Cycle: 2018 Compliance

@anchapin -- should I put filtering in specific instances that aren't working in a separate issue?

For example, here is a field from Tax Lot, in the Property List view Image

There are 7 records without a value in that field Image

If I filter using "" to see the blank records, the program returns one record that is not blank instead of 7 records that are blank. Image

RDmitchell avatar Feb 04 '23 00:02 RDmitchell

I was trying to recreate locally.

At first I was able to recreate. After some investigation I realized it wasn't working because there was no PropertyTaxLots linking the properties and taxlots - which is crucial to this functionality. I was shocked because I didn't realize you could have related properties and taxlots without ProperyTaxLots.

I cleared the db and tried to figure out how I got it in that state to begin with, but I couldn't manage to import related properties and taxlots with the codebase also creating PropertyTaxlots, and consequently, the bug disappearing.

@nllong - Can properties somehow have a taxlot column without a related PropertyTaxlot or taxlot? Do we have any reason to believethis orgs property/taxlot/PropertyTaxlot relationships are atypical?

haneslinger avatar Feb 06 '23 18:02 haneslinger

Instance: staging SHA: 666c8f302 Org: LBNL Test 200 Cycle: 2018 Compliance Cycle

  • Property tab:
    • WORKS: Can filter on a tax lot field, ie, Address, and then add labels to those records without errors
    • WORKS Can filter on Jurisdiction Tax Lot for a value that has multiple instances within a record without errors
    • DOES NOT WORK: Filtering on Jurisdiction Tax Lot ID for "" (blank) or !"" (not blank) fields DOES NOT WORK -- will make a new issue
    • WORKS: Filtering on PM Property ID (property field) for for "" (blank) or !"" (not blank) fields does work
    • DOES NOT WORK: Filtering on ; (semicolon) in Jurisdiction Tax Lot ID (to see records with multiple tax lots) returns no records, when it should return 3
    • DOES NOT WORK: Filtering on Tax Lot field Parcel Owner Telephone for "" results in 0 records instead of 4

@nllong / @anchapin / @haneslinger -- should I make a new issue for the things that aren't working, which is mostly filtering for blank / not blank and semicolon issues on Tax Lot fields on the Property tab (and presumably also Property fields on the Tax Lot tab)

RDmitchell avatar Mar 10 '23 02:03 RDmitchell

Instance: dev1 SHA: 8dc72bf0e Org: LBNL Test 200

  • Property tab:

    • WORKS:
      • Can filter on a tax lot field, ie, Address, and then add labels to those records without errors
      • Can filter on Jurisdiction Tax Lot for a value that has multiple instances within a record without errors
      • Filtering on Jurisdiction Tax Lot ID (and other Tax Lot fields) for !"" shows not blank records
      • Filtering on PM Property ID (property field) for for "" (blank) or !"" (not blank) fields does work
    • DOES NOT WORK:
      • Filtering on Jurisdiction Tax Lot ID (and other Tax Lot fields) for "" (blank) -- it shows no records
      • Can't sort Tax Lot fields
      • Filtering on ; (semicolon) in Jurisdiction Tax Lot ID (to see records with multiple tax lots) returns no records, when it should return 3
        • This is useful so that you can see all the records that have more than one Tax Lot associated with a property
  • Tax Lot tab

    • WORKS:
      • Can filter on a tax lot field, ie, Address, and then add labels to those records without errors
      • Can filter on Jurisdiction Tax Lot with correct results
      • Can sort on tax lot fields such as Jurisdication Tax Lot ID
    • DOES NOT WORK:
      • Filtering on Jurisdiction Tax Lot ID for "" (blank) or !"" (not blank) fields
      • Sort on Property Fields
      • Filtering on ; (semicolon) in PM Property ID (to see records with multiple tax lots) returns no records, when it should
        • This is useful so that you can see all the records that have more than one property associated with a tax lot

RDmitchell avatar Mar 31 '23 02:03 RDmitchell

@isalanglois / @nllong -- should I make a new issues for 2023 with the things that still aren't working for this feature, add it to the Q3 project, then close this one (and I could reference it from the new one)?

RDmitchell avatar Apr 11 '23 22:04 RDmitchell

I have moved the "DOES NOT WORK" issues from the end of this issue into a new issue for the Q3 project #3986

RDmitchell avatar Apr 19 '23 19:04 RDmitchell