Is it a bug in mm-iface-modem.c at setting up carrier config?

Hey

I wanted to open MR for this, but there is an error when I tried to fork project, so I will post it here.

I was just looking at code in mm-iface-modem.c file at interface_initialization_step function at INITIALIZATION_STEP_SETUP_CARRIER_CONFIG case (line 6035) (commit 3aef62c5).

Isn't !mm_base_sim_is_esim_without_profiles (sim) should be mm_base_sim_is_esim_without_profiles (sim) ? Because in current code it seems that we actually skip this step if eSIM HAS profiles. Which I assume incorrect?

            /* If we have a SIM object, and carrier config switching is supported,
             * validate whether we're already using the best config or not. */
            if (!sim)
                mm_obj_dbg (self, "not setting up carrier config: SIM not found");
            else if (!mm_base_sim_is_esim_without_profiles (sim))
                mm_obj_dbg (self, "not setting up carrier config: eSIM without profiles");
            else if (!carrier_config_mapping)
                mm_obj_dbg (self, "not setting up carrier config: mapping file not configured");
            else {
                const gchar *imsi;

                imsi = mm_gdbus_sim_get_imsi (MM_GDBUS_SIM (sim));