Fix minor bug affecting predict and margins

Before, if you ran predict or margins after ppmlhdfe WITHOUT an absorb() option, you would trigger a syntax error due due repeated "if" clauses ("replace .. = .. if .. if ..")

This has been fixed

Author: sergiocorreia

src/ppmlhdfe.ado

@@ -1,4 +1,4 @@
-*! version 2.3.3 02nov2025
+*! version 2.3.4 11jan2025
 *! Authors: Sergio Correia, Paulo Guimarães, Thomas Zylkin
 *! URL: https://github.com/sergiocorreia/ppmlhdfe
 

src/ppmlhdfe.pkg

@@ -19,9 +19,12 @@ d KW: panel data
 d KW: separation
 d
 d Requires: Stata version 13.1
-d Required packages: ftools, reghdfe
 d
-d Author: Sergio Correia. Board of Governors of the Federal Reserve System
+d Required packages:
+d 	ftools
+d 	reghdfe
+d
+d Author: Sergio Correia. Federal Reserve Bank of Richmond
 d Support: sergio.correia@gmail.com
 d
 d Author: Paulo Guimarães. Banco de Portugal
@@ -30,7 +33,7 @@ d
 d Author: Thomas Zylkin. University of Richmond
 d Support: tzylkin@richmond.edu
 d
-d Distribution-Date: 20251102
+d Distribution-Date: 20260111
 d
 f ppmlhdfe.ado
 f ppmlhdfe_header.ado

src/ppmlhdfe.sthlp

@@ -1,5 +1,5 @@
 {smcl}
-{* *! version 2.3.3 02nov2025}{...}
+{* *! version 2.3.4 11jan2025}{...}
 {vieweralsosee "[R] poisson" "help poisson"}{...}
 {vieweralsosee "[R] xtpoisson" "help xtpoisson"}{...}
 {vieweralsosee "[R] glm" "help glm"}{...}

src/ppmlhdfe_p.ado

@@ -32,7 +32,7 @@ program define ppmlhdfe_p
 
 	loc opt `mu' `nondefault'
 	loc y `e(depvar)'
-	loc ifin `"`if' `in'"'
+	loc ifin `"`in' `if'"' // put -in- first so we can add an -if- clause later in some cases
 
 	* Default option is mu
 	if ("`opt'" == "") {
@@ -68,7 +68,10 @@ program define ppmlhdfe_p
 		la var `varlist' "Linear prediction: xb + d[`e(absvars)']"
 	}
 	else {
-		qui replace `varlist' = . if mi(`e(d)') `ifin'
+		* only include "if" when -if- is empty (we can't test against -ifin- because -in- can exist)
+		loc if_literal = cond(`"`if'"'=="", "if", "&")
+		di as error `"qui replace `varlist' = . `ifin' `if_literal' mi(`e(d)') "'
+		qui replace `varlist' = . `ifin' `if_literal' mi(`e(d)') 
 		la var `varlist' "Linear prediction: xb"
 	}