Paths

Table of Contentst

Differential D8256

Break some of Aphlict into reasonable classes with sensible responsibilities
ClosedPublic
Actions

Authored by epriestley on Feb 17 2014, 5:04 AM.

Tags

None

Referenced Files

	F14813701: D8256.diff
	Mon, Jan 27, 1:03 PM

	Unknown Object (File)
	Thu, Jan 23, 3:37 PM

	Unknown Object (File)
	Wed, Jan 22, 11:31 AM

	Unknown Object (File)
	Wed, Jan 22, 4:21 AM

	Unknown Object (File)
	Wed, Jan 22, 4:21 AM

	Unknown Object (File)
	Wed, Jan 22, 4:20 AM

	Unknown Object (File)
	Wed, Jan 22, 3:08 AM

	Unknown Object (File)
	Fri, Jan 17, 7:34 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T4324: Change notification server so it only alerts appropriate clients

Commits

Restricted Diffusion Commit
rP28fe44da0a8c: Break some of Aphlict into reasonable classes with sensible responsibilities

Summary

Ref T4324.

Create Listener to represent listening clients.
Create ListenerList to represent the current list of clients.
Create Logfile to handle logging.

Test Plan

Clicked "Send Test Notification", verified logs, status and notifications all work correctly.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Feb 17 2014, 7:17 PM

Closed by commit rP28fe44da0a8c (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

resources/

celerity/

2 lines

support/

aphlict/

server/

aphlict_server.js

92 lines

lib/

AphlictIDGenerator.js

AphlictListener.js

39 lines

AphlictListenerList.js

57 lines

52 lines

webroot/

rsrc/

externals/

javelin/

core/

24 lines

Diff 19669

resources/celerity/map.php

Loading...

support/aphlict/server/aphlict_server.js

Loading...

support/aphlict/server/lib/AphlictIDGenerator.js

Loading...

support/aphlict/server/lib/AphlictListener.js

Loading...

support/aphlict/server/lib/AphlictListenerList.js

Loading...

support/aphlict/server/lib/AphlictLog.js

Loading...

webroot/rsrc/externals/javelin/core/init_node.js

Loading...