refactor(php): carry the non-zero retry interval types

The consumer builder takes NonZeroIggyDuration for the retry intervals, so the
binding's own zero check hands that type over.

Author: ethanlin01x

foreign/php/src/client.rs

@@ -343,9 +343,8 @@ impl IggyClient {
             builder = builder.auto_commit(auto_commit.into());
         }
         builder = match poll_interval_micros {
-            Some(micros) => {
-                builder.poll_interval(non_zero_duration_micros("poll_interval_micros", micros)?)
-            }
+            Some(micros) => builder
+                .poll_interval(non_zero_duration_micros("poll_interval_micros", micros)?.get()),
             None => builder.without_poll_interval(),
         };
         if let Some(micros) = polling_retry_interval_micros {
@@ -408,12 +407,7 @@ impl IggyClient {
     }
 }
 
-fn non_zero_duration_micros(field: &str, micros: u64) -> PhpResult<IggyDuration> {
-    if micros == 0 {
-        return Err(to_php_exception(format!(
-            "'{field}' must be greater than 0 microseconds"
-        )));
-    }
-
-    Ok(IggyDuration::from(micros))
+fn non_zero_duration_micros(field: &str, micros: u64) -> PhpResult<NonZeroIggyDuration> {
+    NonZeroIggyDuration::try_from(micros)
+        .map_err(|_| to_php_exception(format!("'{field}' must be greater than 0 microseconds")))
 }