Merge branch 'dev' into 'dev'

Breast fullness is not displayed and the "milk yourself" action is not invoked on a pawn when the rjw-mc-biotech mod is present

See merge request lutepickle/rjw_menstruation!6

1.4/source/RJW_Menstruation/RJW_Menstruation/Utility.cs

@@ -276,7 +276,10 @@ namespace RJW_Menstruation
             float res = 0;
             if (VariousDefOf.Hediff_Heavy_Lactating_Permanent != null)
             {
-                if (pawn.health.hediffSet.HasHediff(VariousDefOf.Hediff_Heavy_Lactating_Permanent)) milkcomp = pawn.AllComps.FirstOrDefault(x => x.GetType().ToString().ToLower().Contains("hypermilkable"));
+                if (pawn.health.hediffSet.HasHediff(VariousDefOf.Hediff_Heavy_Lactating_Permanent)
+                    || pawn.health.hediffSet.HasHediff(VariousDefOf.Hediff_Lactating_Permanent)
+                    || pawn.health.hediffSet.HasHediff(VariousDefOf.Hediff_Lactating_Natural)
+                    || pawn.health.hediffSet.HasHediff(VariousDefOf.Hediff_Lactating_Drug)) milkcomp = pawn.AllComps.FirstOrDefault(x => x.GetType().ToString().ToLower().Contains("milkablehuman"));
                 else milkcomp = pawn.AllComps.FirstOrDefault(x => x.GetType().ToString().ToLower().Contains("milkable"));
             }
             else

1.4/source/RJW_Menstruation/RJW_Menstruation/VariousDefOf.cs

@@ -158,7 +158,7 @@ namespace RJW_Menstruation
         public static readonly HediffDef Hediff_Lactating_Natural = DefDatabase<HediffDef>.GetNamedSilentFail("Lactating_Natural");
         public static readonly HediffDef Hediff_Lactating_Permanent = DefDatabase<HediffDef>.GetNamedSilentFail("Lactating_Permanent");
         public static readonly HediffDef Hediff_Heavy_Lactating_Permanent = DefDatabase<HediffDef>.GetNamedSilentFail("Heavy_Lactating_Permanent");
-        public static readonly JobDef Job_LactateSelf_MC = DefDatabase<JobDef>.GetNamedSilentFail("LactateSelf_MC");
+        public static readonly JobDef Job_LactateSelf_MC = DefDatabase<JobDef>.GetNamedSilentFail("MilkSelf");
 
         // Defs from Sexperience Ideology
         public static readonly PreceptDef Pregnancy_Elevated = DefDatabase<PreceptDef>.GetNamedSilentFail("Pregnancy_Elevated");

1.5/source/RJW_Menstruation/RJW_Menstruation/HediffComps/HediffComp_Menstruation.cs

@@ -639,10 +639,14 @@ namespace RJW_Menstruation
             if (Scribe.mode == LoadSaveMode.PostLoadInit)
             {
                 Initialize();
-                if (pregnancy is HediffWithParents vanillaPreg && vanillaPreg.Mother == vanillaPreg.Father)   // Fix mother == father issue
+                // Biotech labor errors if the mother is also the father, so don't let that happen
+                if (pregnancy is HediffWithParents vanillaPreg && vanillaPreg.Mother == vanillaPreg.Father && vanillaPreg.Mother != null)
+                {
+                    Log.Warning($"Pregnancy of {Pawn} has same mother and father, setting father to null");
                     vanillaPreg.SetParents(vanillaPreg.Mother, null, vanillaPreg.geneSet);
                 }
             }
+        }
 
 
         public override void CompPostPostAdd(DamageInfo? dinfo)
@@ -790,7 +794,7 @@ namespace RJW_Menstruation
         {
             get
             {
-                if (Pawn.Dead) return null;
+                if (Pawn.Dead || !Pawn.ShouldCycle()) return null;
                 StringBuilder tip = new StringBuilder();
                 tip.Append(Translations.Dialog_WombInfo01);
                 tip.Append(": ");
@@ -812,7 +816,11 @@ namespace RJW_Menstruation
 
         public override string CompDebugString()
         {
-            if (Pawn.Dead || curStage == Stage.None || curStage == Stage.Infertile || curStage == Stage.Pregnant) return null;
+            if (Pawn.Dead ||
+                !Pawn.ShouldCycle() ||
+                curStage == Stage.None ||
+                curStage == Stage.Infertile ||
+                curStage == Stage.Pregnant) return null;
             StringBuilder debugString = new StringBuilder();
             debugString.Append($"Time to next state: ");
             debugString.Append(GenDate.ToStringTicksToPeriod(TicksToNextStage()));

1.5/source/RJW_Menstruation/RJW_Menstruation/HediffComps/MenstruationUtility.cs

@@ -242,7 +242,7 @@ namespace RJW_Menstruation
             bool isInduced = comp is HediffComp_InducedOvulator;
             if (comp.curStage == HediffComp_Menstruation.Stage.Follicular &&
                 isInduced &&
-                comp.Pawn.jobs.curDriver is JobDriver_Sex job &&
+                comp.Pawn.jobs?.curDriver is JobDriver_Sex job &&
                 job.Sexprops != null &&
                 !UsingCondom(comp.Pawn, job.Partner) &&
                 (job.Sexprops.sexType == xxx.rjwSextype.Vaginal || job.Sexprops.sexType == xxx.rjwSextype.DoublePenetration))

1.5/source/RJW_Menstruation/RJW_Menstruation/Patch/Pawn_Patch.cs

@@ -33,10 +33,14 @@ namespace RJW_Menstruation
 
             foreach (LocalTargetInfo t in selftargets)
             {
-                if (t.Pawn == pawn && pawn.HasMenstruationComp()) opts.AddDistinct(MakeSelfMenu(pawn, t));
+                if (t.Pawn == pawn)
+                {
+                    if (pawn.HasMenstruationComp() && pawn.ShouldCycle())
+                        opts.AddDistinct(MakeSelfMenu(pawn, t));
                     break;
                 }
             }
+        }
 
         public static FloatMenuOption MakeSelfMenu(Pawn pawn, LocalTargetInfo target)
         {