Conpherence - fix fatal on list view for 100+ messages
ClosedPublic
Actions

Authored by btrahan on Nov 6 2014, 1:03 AM.

Tags

None

Referenced Files

	F14759769: D10793.id25903.diff
	Wed, Jan 22, 2:06 PM

	Unknown Object (File)
	Tue, Jan 21, 9:21 AM

	Unknown Object (File)
	Sat, Jan 18, 2:32 AM

	Unknown Object (File)
	Mon, Jan 13, 12:04 AM

	Unknown Object (File)
	Mon, Dec 30, 5:31 PM

	Unknown Object (File)
	Dec 24 2024, 2:58 AM

	Unknown Object (File)
	Dec 22 2024, 2:45 AM

	Unknown Object (File)
	Dec 16 2024, 6:00 PM

View All Files

Subscribers

epriestley

Korvin

Tokens

"Haypence" token, awarded by chad.

Details

Reviewers

epriestley

Maniphest Tasks

T6480: Conpherence list view fatals

Commits

Restricted Diffusion Commit
rP351f4722d960: Conpherence - fix fatal on list view for 100+ messages

Summary

Fixes T6480. @epriestley is very popular and has over 100 messages, thus triggering this issue. fix the typo because bugs are bad.

Test Plan

set limit to 1, observed fatal, applied patch and fatal went away

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 25902.Nov 6 2014, 1:03 AM

btrahan added a task: T6480: Conpherence list view fatals.

btrahan retitled this revision from to Conpherence - fix fatal on list view for 100+ messages.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

btrahan added a subscriber: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptNov 6 2014, 1:03 AM

Harbormaster completed remote builds in B2958: Diff 25902.Nov 6 2014, 1:04 AM

everyone loves me :3

This revision is now accepted and ready to land.Nov 6 2014, 1:30 AM

Closed by commit rP351f4722d960: Conpherence - fix fatal on list view for 100+ messages (authored by btrahan, committed by btrahan). · Explain WhyNov 6 2014, 1:32 AM

This revision was automatically updated to reflect the committed changes.

chad awarded a token.Nov 6 2014, 2:42 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceListController.php

2 lines

Diff 25903

View Options

src/applications/conpherence/controller/ConpherenceListController.php

Conpherence - fix fatal on list view for 100+ messagesClosedPublicActions