Properties such as 'errors' and 'module' are getting exported. They should not be.

Files: 

Comments

'module' is intentionally exported:

      // Even though Flag adds the module name itself later, we add the module
      // name here for reference by other modules (such as Features).
      $new_flag['module'] = $module;

Status:Active» Needs review
StatusFileSize
new1.42 KB
PASSED: [[SimpleTest]]: [MySQL] 266 pass(es).
[ View ]

Figured this bit of the code could use a little clean-up at the same time.

Status:Needs review» Reviewed & tested by the community
StatusFileSize
new816 bytes
new783 bytes

I have tested this now and I can confirm it works. I have attached the before and after versions of the export. The original exported the 'errors' array. However, after the patch was applied, the 'errors' array, as expected, was not exported.

Status:Reviewed & tested by the community» Fixed

Thanks for the review!

Committed.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.