Fixed
The previous version was redundant
--------------------------------------------------------------------------------
Fixed again
--------------------------------------------------------------------------------
Thanks for sharing this patch!

I have updated it to apply cleanly to 1.1-stable.
--------------------------------------------------------------------------------
I do appreciate Yuki Kita and David Escala work.
I think this feature is very useful for security audit in an organization. Probably there are so many people who want this feature.

I have updated the patch:

* Compatible with latest trunk (r16680)
* Add status to field.
* Remove custom_fields from export contents because it take very long time to export thousands users.
* Add tests.

Link to export CSV
!{width: 70%; border:1px solid #ccc}users_csv_link.png!

exported CSV
!{width: 70%; border:1px solid #ccc}users_csv.png!
--------------------------------------------------------------------------------
The patch works fine and passed all test. Looks good to me.
I think 3.4.0 is feature freezing now, so setting target version to 3.5.0.
--------------------------------------------------------------------------------
From the list of users page in Admin, you can just right click and save as HTML.

Then open the file in excel and extract the user data you need.

It's the easy way, and I figured it out because the patch was not working for me.

~ Brian

--------------------------------------------------------------------------------
Brian Schwartz, Thank you for trying users_export5.patch.
I fixed users_export5.patch to work on latest trunk (r17022)
--------------------------------------------------------------------------------
Mizuki, maybe you can simplify the patch because array constant LABEL_BY_STATUS was introduced by #15201 so your patch does not need to introduce @STATUS_NAMES@ and rewrite @users_status_options_for_select@, @css_classes@ and @users_status_options_for_select@ method.
--------------------------------------------------------------------------------
Go MAEDA wrote:
> Mizuki, maybe you can simplify the patch because array constant LABEL_BY_STATUS was introduced by #15201 so your patch does not need to introduce @STATUS_NAMES@ and rewrite @users_status_options_for_select@, @css_classes@ and @users_status_options_for_select@ method.

Thank you for telling me the new change information.
I fixed the patch.

In addition, I also made changes according to #26279.

--------------------------------------------------------------------------------
Committed. thank you all to those who contributed to this issue.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

relates,Closed,33126,Support custom fields when exporting users to CSV