php-imap icon indicating copy to clipboard operation
php-imap copied to clipboard

[Feature Request] Add bool return to setFlag(), markMailAsRead(), etc.

Open loekvangool opened this issue 3 years ago • 2 comments

Is your feature request related to a problem? Please describe. setFlag() and its invokers like markMailAsRead() return void at the moment, can they instead return a bool?

loekvangool avatar Mar 03 '23 09:03 loekvangool

Question is if it is necessary because when there is something wrong when you try set or clear the flag, you receive UnexpectedValueException

michalcharvat avatar Mar 16 '23 17:03 michalcharvat

Thanks for the reply. Saw that, however writing try/catch blocks is IMO quite cumbersome to implement, versus return values.

I do understand that it is a style difference, not right/wrong.

loekvangool avatar Mar 16 '23 17:03 loekvangool