fix: issue #9876 import spreadsheet template with mandatory fields

[himanshisrestha]

What

This solves the issue #9876 import spreadsheet template with mandatory fields only on the producers platform.
In the import template generated by https://world.pro.openfoodfacts.org/cgi/generate_sample_import_file.pl , added a parameter "?fields=mandatory" to output the excel file with just the mandatory fields.
The issue was solved using param() function of the CGI.pm module. Before this, the code was setup to just find the importance of the header cell and then apply styling accordingly to write the cell. Using the param() function, the passed argument "fields=mandatory" was accepted as a name=value pair which made it easier to decide whether to add the the cell or not.
The change can be seen in the image attached, which returns an excel sheet consisting of 28 columns of mandatory fields only.

Screenshot

Related issue(s) and discussion

• Fixes Generate an import spreadsheet template with mandatory fields only on the producers platform #9876

[stephanegigandet]

This is good for testing, but it should not be included in the PR.

[alexgarel]

@himanshisrestha, when you test, use stephane or tacinte account to avoid having to change this file.

[himanshisrestha]

@alexgarel could you please guide me on how to use it

[alexgarel]

you just go to sign-in -- create account and use the username "stephane"

[stephanegigandet]

This should not be included in the PR

[alexgarel]

@himanshisrestha if you use linux or mac, you might use direnv

[stephanegigandet]

Hello @himanshisrestha , thank you for the PR!

I made some comments. In addition, there are some code that was changed that we want to keep:

my $description = lang("description");
my $field_id;
my $comment;

$worksheet->set_row(0, 70);
$worksheet->set_column('A:ZZ', 30);
$worksheet->set_column('A:A', 30);
$worksheet->write($description_row, 0, $description, $formats{'description'});

This is needed in order to have "Description" translated in other languages.

Could you change the PR so that it contains only the one change for keeping only the mandatory fields? Thanks!

[stephanegigandet]

Also you can run "make lint" to automatically format the code.

[himanshisrestha]

Also you can run "make lint" to automatically format the code.

Yes, I can. I'm changing all what you mentioned and will update you soon.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[stephanegigandet]

I think this change is unrelated, could you remove it from the PR?

[alexgarel]

this is the last empty line which was removed. It sometimes depends on the IDE.

[stephanegigandet]

Looks good to me! (except for the modification in .env file)

Thank you!

[himanshisrestha]

Happy to mark my contribution ☺️

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.52%. Comparing base (dc04d18) to head (578e938).
Report is 124 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9934      +/-   ##
==========================================
- Coverage   49.54%   49.52%   -0.02%     
==========================================
  Files          67       71       +4     
  Lines       20650    20899     +249     
  Branches     4980     5019      +39     
==========================================
+ Hits        10231    10351     +120     
- Misses       9131     9254     +123     
- Partials     1288     1294       +6     

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