Closed Bug 305941 Opened 20 years ago Closed 19 years ago

Remove refreshed_when from profiles and last_changed from groups

Categories

(Bugzilla :: User Accounts, defect)

2.21
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: bugreport, Assigned: LpSolit)

Details

Attachments

(1 file, 2 obsolete files)

They are no longer needed.
Assignee: general → user-accounts
Component: Bugzilla-General → User Accounts
Target Milestone: --- → Bugzilla 2.22
The trunk is now frozen to prepare Bugzilla 2.22. Enhancement bugs are retargetted to 2.24.
Target Milestone: Bugzilla 2.22 → Bugzilla 2.24
groups.last_changed might actually be useful if we exposed it in the UI in editgroups for example.
Severity: enhancement → normal
(In reply to comment #2) > groups.last_changed might actually be useful if we exposed it in the UI in > editgroups for example. > We actually have no UI for it. Do we plan to implement it? We cannot remove this column now and add it again later; we would probably break checksetup.pl.
Attached patch patch, v1 (obsolete) — Splinter Review
Assignee: user-accounts → LpSolit
Status: NEW → ASSIGNED
Attachment #232197 - Flags: review?(mkanat)
Comment on attachment 232197 [details] [diff] [review] patch, v1 It's bz_drop_column, not drop_column.
Attachment #232197 - Flags: review?(mkanat) → review-
Attached patch patch, v1.1 (obsolete) — Splinter Review
s/drop_column/bz_drop_column/
Attachment #232197 - Attachment is obsolete: true
Attachment #232198 - Flags: review?(mkanat)
Attachment #232198 - Attachment is obsolete: true
Attachment #232200 - Flags: review?(mkanat)
Attachment #232198 - Flags: review?(mkanat)
Comment on attachment 232200 [details] [diff] [review] unbitrotten patch, v1.2 Yeah, looks good. I ran a few tests, and I checked that last_changed and refreshed_when are totally gone from the code, and everything is good.
Attachment #232200 - Flags: review?(mkanat) → review+
Flags: approval?
Flags: approval? → approval+
Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.537; previous revision: 1.536 done Checking in editgroups.cgi; /cvsroot/mozilla/webtools/bugzilla/editgroups.cgi,v <-- editgroups.cgi new revision: 1.78; previous revision: 1.77 done Checking in editproducts.cgi; /cvsroot/mozilla/webtools/bugzilla/editproducts.cgi,v <-- editproducts.cgi new revision: 1.128; previous revision: 1.127 done Checking in editusers.cgi; /cvsroot/mozilla/webtools/bugzilla/editusers.cgi,v <-- editusers.cgi new revision: 1.130; previous revision: 1.129 done Checking in sanitycheck.cgi; /cvsroot/mozilla/webtools/bugzilla/sanitycheck.cgi,v <-- sanitycheck.cgi new revision: 1.121; previous revision: 1.120 done Checking in Bugzilla/Group.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Group.pm,v <-- Group.pm new revision: 1.14; previous revision: 1.13 done Checking in Bugzilla/User.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v <-- User.pm new revision: 1.123; previous revision: 1.122 done Checking in Bugzilla/DB/Schema.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema.pm,v <-- Schema.pm new revision: 1.59; previous revision: 1.58 done Checking in Bugzilla/Install/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm,v <-- DB.pm new revision: 1.5; previous revision: 1.4 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: