ROB: discard /I in choice fields for compatibility with acrobat

[pubpub-zz]

closes #2611

[stefan6419846]

Wouldn't it be better to set both the /I and /V fields instead?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.93%. Comparing base (6d9a7ec) to head (06b7f1c).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2614      +/-   ##
==========================================
+ Coverage   94.92%   94.93%   +0.01%     
==========================================
  Files          50       50              
  Lines        8316     8318       +2     
  Branches     1667     1668       +1     
==========================================
+ Hits         7894     7897       +3     
  Misses        261      261              
+ Partials      161      160       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pubpub-zz]

/I should only be used for multiple choice. here Acrobat is using it on a single selection field. It does not respect the specification.
I thought about setting both but it will complexify the code to check for free text entry in a combo box, multiple entries and so one, where as the spec is clear about the fact that /V has priority.

[chrischanwz]

Hi, I tracked this issue from #2611 , where i was facing the same issue of not being able to fill dropdown fields (text fields are alright). i installed the latest dev ver 4.3.0, but it seems to not make a difference. can I check what changes have been made to the update_page_form_field_values method and what I am supposed to do to fill up the Dropdown fields? I greatly appreciate any help, thank you very much
currently my code looks something like this:

def handle_bom(value):
    if isinstance(value, str) and value.startswith(("\xfe\xff", "\xff\xfe")):
        value = value.encode('latin1').decode('utf-16')
    return value

# other code omitted
with open(pdf_file_base, "rb") as file:
        reader = PdfReader(file)
        writer = PdfWriter()
        writer.append(reader)  

        for i, (index, row) in enumerate(group.iterrows()):   # i am extracting data from excel sheet basically
             data_to_fill = row.to_dict()
             data_to_fill = {key: handle_bom(str(value)) for key, value in data_to_fill.items()}  # i needed to do this to solve AttributeError: 'float object has no attribute 'startswith' 
             page = writer.pages[i] 
             writer.update_page_form_field_values(page, data_to_fill, auto_regenerate=True,)

[pubpub-zz]

@chrischanwz
Can you please open a new issue to ease handling with code, pdf input and output to ease analysis

[stefan6419846]

For the changes: Just have a look at the diff of this PR: https://github.com/py-pdf/pypdf/pull/2614/files

For further analysis, please open a new issue as already pointed out, which includes a standalone example we can reproduce.