We're updating the issue view to help you get more done. 

Change default behavior of Email on Away plugin

Description

In 1.0.1 version of this plugin default behavior has been accidentally changed to hide the email address (because if there is no system property it is treated as a false value). Changing that, so old behavior would be as default and old users won't see difference and won't have to change anything.

Environment

None

Acceptance Test - Entry

None

Activity

Show:
wroot
April 10, 2016, 10:10 AM

The way how Openfire operates with system properties is very confusing. When there is no property, the check works as false. But next time it doesn't. Will have to think again how to make it work as before by default.

wroot
April 10, 2016, 11:14 AM

I was wrong. When there is no such property, getBooleanProperty returns true, not false (as if the property would be there with true value, only false itself returns false). Will submit another PR.

wroot
April 10, 2016, 11:30 AM

Submitted https://github.com/igniterealtime/Openfire/pull/574 (also bumped the version again to make it update properly).

Fixed

Assignee

wroot

Reporter

wroot

Labels

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Minor
Configure