phorge-phorge/src/infrastructure/markup/rule/PhabricatorRemarkupRuleDifferential.php

<?php

/**
 * @group markup
 */
final class PhabricatorRemarkupRuleDifferential
  extends PhabricatorRemarkupRuleObjectName {

  protected function getObjectNamePrefix() {
    return 'D';
  }

}
Link "DNNNN" and "TNNNN" in Phabricator remarkup. Summary: Autolink Differential and Maniphest objects. Test Plan: Typed "D12345" and "T12345" into the Differential comment preview, got links. Typed "http://www.elsewhere.com/D12345" and got a single link to that URI, not a mess where the D12345 part linked incorrectly. Reviewers: aran CC: Differential Revision: 35 2011-02-11 18:06:43 -08:00			`<?php`

			`/**`
			`* @group markup`
			`*/`
Add "final" to (almost) everything else Summary: Last of the big final patches. Left a few debatable classes (12 out of about 400) that I'll deal with individually eventually. Test Plan: Ran testEverythingImplemented. Reviewers: btrahan Reviewed By: btrahan CC: aran, epriestley Maniphest Tasks: T795 Differential Revision: https://secure.phabricator.com/D1881 2012-03-13 16:21:04 -07:00			`final class PhabricatorRemarkupRuleDifferential`
Add comment linking to Maniphest and Differential Summary: Allows you to link to comments with "D123#3" or "T123#3", then adds a pile of JS to try to make it not terrible. :/ The thing I'm trying to avoid here is when someone says "look at this! http://blog.com/#comment-239291" and you click and your browser jumps somewhere random and you have no idea which comment they meant. Since I really hate this, I've tried to avoid it by making sure the comment is always highlighted. Test Plan: Put T1#1 and D1#1 in remarkup and verified they linked properly. Clicked anchors on individual comments. Faked all comments hidden in Differential and verified they expanded on anchor or anchor change. Reviewed By: aran Reviewers: aran, tomo, mroch, jungejason, tuomaspelkonen CC: aran, epriestley Differential Revision: 383 2011-05-31 10:23:31 -07:00			`extends PhabricatorRemarkupRuleObjectName {`
Link "DNNNN" and "TNNNN" in Phabricator remarkup. Summary: Autolink Differential and Maniphest objects. Test Plan: Typed "D12345" and "T12345" into the Differential comment preview, got links. Typed "http://www.elsewhere.com/D12345" and got a single link to that URI, not a mess where the D12345 part linked incorrectly. Reviewers: aran CC: Differential Revision: 35 2011-02-11 18:06:43 -08:00
Add comment linking to Maniphest and Differential Summary: Allows you to link to comments with "D123#3" or "T123#3", then adds a pile of JS to try to make it not terrible. :/ The thing I'm trying to avoid here is when someone says "look at this! http://blog.com/#comment-239291" and you click and your browser jumps somewhere random and you have no idea which comment they meant. Since I really hate this, I've tried to avoid it by making sure the comment is always highlighted. Test Plan: Put T1#1 and D1#1 in remarkup and verified they linked properly. Clicked anchors on individual comments. Faked all comments hidden in Differential and verified they expanded on anchor or anchor change. Reviewed By: aran Reviewers: aran, tomo, mroch, jungejason, tuomaspelkonen CC: aran, epriestley Differential Revision: 383 2011-05-31 10:23:31 -07:00			`protected function getObjectNamePrefix() {`
			`return 'D';`
Link "DNNNN" and "TNNNN" in Phabricator remarkup. Summary: Autolink Differential and Maniphest objects. Test Plan: Typed "D12345" and "T12345" into the Differential comment preview, got links. Typed "http://www.elsewhere.com/D12345" and got a single link to that URI, not a mess where the D12345 part linked incorrectly. Reviewers: aran CC: Differential Revision: 35 2011-02-11 18:06:43 -08:00			`}`

			`}`