fix: processRoomAbandonment callback not processing data correctly

[KevLehman]

Proposed changes (including videos or screenshots)

• Prevent callback from failing when BH configuration doesn't include all fields
• Fix internal callback logic to properly calculate data

Issue(s)

https://rocketchat.atlassian.net/browse/CORE-619

Steps to test or reproduce

Further comments

[dionisio-bot]

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

[changeset-bot]

🦋 Changeset detected

Latest commit: 11fbb28

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 32 packages

Name	Type
@rocket.chat/meteor	Patch
@rocket.chat/core-typings	Patch
@rocket.chat/rest-typings	Patch
@rocket.chat/apps	Patch
@rocket.chat/core-services	Patch
@rocket.chat/cron	Patch
@rocket.chat/fuselage-ui-kit	Patch
@rocket.chat/gazzodown	Patch
@rocket.chat/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/ui-contexts	Patch
@rocket.chat/account-service	Patch
@rocket.chat/authorization-service	Patch
@rocket.chat/ddp-streamer	Patch
@rocket.chat/omnichannel-transcript	Patch
@rocket.chat/presence-service	Patch
@rocket.chat/queue-worker	Patch
@rocket.chat/stream-hub-service	Patch
@rocket.chat/api-client	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/license	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/presence	Patch
rocketchat-services	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/models	Patch
@rocket.chat/ui-avatar	Patch
@rocket.chat/ui-client	Patch
@rocket.chat/ui-video-conf	Patch
@rocket.chat/web-ui-registration	Patch
@rocket.chat/instance-status	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codecov]

Codecov Report

Attention: Patch coverage is 92.85714% with 3 lines in your changes missing coverage. Please review.

Project coverage is 59.44%. Comparing base (d828b44) to head (11fbb28).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #33036      +/-   ##
===========================================
+ Coverage    59.40%   59.44%   +0.03%     
===========================================
  Files         2543     2544       +1     
  Lines        63180    63251      +71     
  Branches     14219    14240      +21     
===========================================
+ Hits         37532    37598      +66     
  Misses       22933    22933              
- Partials      2715     2720       +5     

Flag	Coverage Δ
unit	76.09% <92.85%> (+0.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

[MarcosSpessatto]

I'd suggest to split the 2 fixes you mentioned in 2 different PRs, since we are changing the way things are calculated, this could lead to some unexpected future problems.

[KevLehman]

Changed this PR to another task. Both issues will be fixed anyways as this changes the same code the other PR does.