search

awesomemotive / easy-digital-downloads

Sell digital downloads through WordPress
https://easydigitaldownloads.com
GNU General Public License v2.0
867 stars 473 forks source link

Add Context to EDD Errors #3242

Open chriscct7 opened 9 years ago

chriscct7 commented 9 years ago

We should add support for contexts in EDD errors

pippinsplugins commented 9 years ago

To elaborate a little, this could help us resolve issues where errors for one screen accidentally pollute another screen.

For example, if an error is logged somewhere unrelated while updating a customer, the customer update will detect that error and fail.

cklosowski commented 9 years ago

Are we thinking maybe when you register your error with EDD, you add a context, then when doing edd_has_errors( $context ) can only return the errors if they meet context?

chriscct7 commented 9 years ago

Yeah, and maybe either a view or action parameter (or both) would be nice.

pippinsplugins commented 9 years ago

Yep! Rcp does this via a third parameter in the add error function.

On Friday, March 13, 2015, Chris Christoff notifications@github.com wrote:

Yeah, and maybe either a view or action parameter (or both) would be nice.

— Reply to this email directly or view it on GitHub https://github.com/easydigitaldownloads/Easy-Digital-Downloads/issues/3242#issuecomment-79458708 .

pippinsplugins commented 9 years ago

Punting to 2.6.