fix: Send transition when IKEA bulb is turned OFF when not already OFF Koenkk/zigbee2mqtt#23825

Author: Koenkk

src/converters/toZigbee.ts

@@ -1088,8 +1088,12 @@ const converters2 = {
                 //  'MoveToLevelWithOnOff' despite not supporting the cluster; others, like the LEDVANCE SMART+
                 //  plug, do not.)
                 brightness = transition.specified || brightness === 0 ? 0 : undefined;
-                if (brightness !== undefined && utils.getMetaValue(entity, meta.mapped, 'noOffTransition', {atLeastOnce: true}, false)) {
-                    logger.debug(`Supressing OFF transition since entity has noOffTransition=true`, NS);
+                if (
+                    brightness !== undefined &&
+                    meta.state.state === 'OFF' &&
+                    utils.getMetaValue(entity, meta.mapped, 'noOffTransitionWhenOff', {atLeastOnce: true}, false)
+                ) {
+                    logger.debug(`Supressing OFF transition since entity is OFF and has noOffTransitionWhenOff=true`, NS);
                     brightness = undefined;
                 }
                 if (meta.state.brightness !== undefined && meta.state.state === 'ON') {

src/lib/ikea.ts

@@ -107,7 +107,7 @@ export function ikeaLight(args?: Omit<LightArgs, 'colorTemp'> & {colorTemp?: tru
     // https://github.com/Koenkk/zigbee2mqtt/issues/23863
     result.meta = {
         ...result.meta,
-        noOffTransition: (entity) => {
+        noOffTransitionWhenOff: (entity) => {
             const softwareBuildID = entity.getDevice().softwareBuildID;
             return softwareBuildID && !softwareBuildID.includes('-') && semver.gt(softwareBuildID ?? '0.0.0', '1.0.021', true);
         },

src/lib/types.ts

@@ -222,7 +222,7 @@ export interface DefinitionMeta {
     /**
      * Never use a transition when transitioning to off (even when specified)
      */
-    noOffTransition?: boolean | ((entity: Zh.Endpoint) => boolean);
+    noOffTransitionWhenOff?: boolean | ((entity: Zh.Endpoint) => boolean);
 }
 
 export type Configure = (device: Zh.Device, coordinatorEndpoint: Zh.Endpoint, definition: Definition) => Promise<void>;