Manual synchronization error

I have attached a patch that I worked on for a client site that had this same problem, It is slightly different in that I stuck with using the database but I instantised 2 database arrays as per civicrm entity which provides d9 views integration and modified the database query appropriately. That being said I think with the API or database way it is a bit of 6 of 1 half a dozen of the other, I'll let the maintainers decide which way they want to go.

In created a PR to implement corrections proposed by jonhalle.
I've not tested it on 2 databases (one for drupal, an another for civicrm).

@seamus_lee your patch gives this error: "Class "Drupal\civicrm_member_roles\Database" not found". Need to add use Drupal\Core\Database\Database; I think.

The MR https://git.drupalcode.org/project/civicrm_member_roles/-/merge_requests/2 works for me. The code looks good to me.

sadashiv → closed merge request !2

sadashiv → credited pradpnayak → .

Released in 8.x-1.0-beta3

Marking this as fixed

Automatically closed - issue fixed for 2 weeks with no activity.