perf: Ignore cancelled pick lists while fetching picked items

[ankush]

docstatus!=2 is already a condition but it's not useful in status index. Hence add separate condition to prevent over-reading data.

Before:

+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+-------+---------+----------+-------->
| id   | select_type | table             | type  | possible_keys        | key          | key_len | ref                                 | rows  | r_rows  | filtered | r_filte>
+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+-------+---------+----------+-------->
|    1 | SIMPLE      | tabPick List      | range | PRIMARY,status_index | status_index | 563     | NULL                                | 10591 | 7280.00 |   100.00 |      99>
|    1 | SIMPLE      | tabPick List Item | ref   | parent               | parent       | 563     | _fc28cbacdbac857b.tabPick List.name | 3     | 8.63    |   100.00 |       0>
+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+-------+---------+----------+-------->

After:

+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+------+---------+----------+--------->
| id   | select_type | table             | type  | possible_keys        | key          | key_len | ref                                 | rows | r_rows  | filtered | r_filter>
+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+------+---------+----------+--------->
|    1 | SIMPLE      | tabPick List      | range | PRIMARY,status_index | status_index | 563     | NULL                                | 1549 | 1547.00 |   100.00 |      99.>
|    1 | SIMPLE      | tabPick List Item | ref   | parent               | parent       | 563     | _fc28cbacdbac857b.tabPick List.name | 3    | 7.85    |   100.00 |       4.>
+------+-------------+-------------------+-------+----------------------+--------------+---------+-------------------------------------+------+---------+----------+--------->

[codecov]

Codecov Report

Merging #35737 (fa8bd99) into develop (50f8385) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #35737      +/-   ##
===========================================
- Coverage    64.14%   64.13%   -0.01%     
===========================================
  Files          810      810              
  Lines        61880    61880              
===========================================
- Hits         39690    39689       -1     
- Misses       22190    22191       +1     

Impacted Files	Coverage Δ
erpnext/stock/doctype/pick_list/pick_list.py	80.00% <ø> (ø)

... and 1 file with indirect coverage changes