Page MenuHomePhabricator
Files F15440511

D17875.id43001.diff
No OneTemporary
Actions

D17875.id43001.diff
View Options

diff --git a/src/applications/legalpad/editor/LegalpadDocumentEditEngine.php b/src/applications/legalpad/editor/LegalpadDocumentEditEngine.php
--- a/src/applications/legalpad/editor/LegalpadDocumentEditEngine.php
+++ b/src/applications/legalpad/editor/LegalpadDocumentEditEngine.php
@@ -114,53 +114,53 @@
->setConduitTypeDescription(pht('New document signature type.'))
->setValue($object->getSignatureType())
->setOptions(LegalpadDocument::getSignatureTypeMap())
- ->setTransactionType(
+ ->setTransactionType(
LegalpadDocumentSignatureTypeTransaction::TRANSACTIONTYPE);
- $show_require = true;
- } else {
- $fields[] = id(new PhabricatorStaticEditField())
- ->setLabel(pht('Who Should Sign?'))
- ->setValue($object->getSignatureTypeName());
- $individual = LegalpadDocument::SIGNATURE_TYPE_INDIVIDUAL;
- $show_require = $object->getSignatureType() == $individual;
- }
-
- if ($show_require && $is_admin) {
- $fields[] =
- id(new PhabricatorBoolEditField())
- ->setKey('requireSignature')
- ->setOptions(
- pht('No Signature Required'),
- pht('Signature Required to use Phabricator'))
- ->setAsCheckbox(true)
- ->setTransactionType(
- LegalpadDocumentRequireSignatureTransaction::TRANSACTIONTYPE)
- ->setDescription(pht('Marks this document as required signing.'))
- ->setConduitDescription(
- pht('Marks this document as required signing.'))
- ->setValue($object->getRequireSignature());
- }
-
+ $show_require = true;
+ } else {
+ $fields[] = id(new PhabricatorStaticEditField())
+ ->setLabel(pht('Who Should Sign?'))
+ ->setValue($object->getSignatureTypeName());
+ $individual = LegalpadDocument::SIGNATURE_TYPE_INDIVIDUAL;
+ $show_require = $object->getSignatureType() == $individual;
+ }
+
+ if ($show_require && $is_admin) {
$fields[] =
- id(new PhabricatorRemarkupEditField())
- ->setKey('preamble')
- ->setLabel(pht('Preamble'))
- ->setDescription(pht('The preamble of the document.'))
- ->setConduitTypeDescription(pht('New document preamble.'))
- ->setValue($object->getPreamble())
+ id(new PhabricatorBoolEditField())
+ ->setKey('requireSignature')
+ ->setOptions(
+ pht('No Signature Required'),
+ pht('Signature Required to use Phabricator'))
+ ->setAsCheckbox(true)
->setTransactionType(
- LegalpadDocumentPreambleTransaction::TRANSACTIONTYPE);
+ LegalpadDocumentRequireSignatureTransaction::TRANSACTIONTYPE)
+ ->setDescription(pht('Marks this document as required signing.'))
+ ->setConduitDescription(
+ pht('Marks this document as required signing.'))
+ ->setValue($object->getRequireSignature());
+ }
- $fields[] =
- id(new PhabricatorRemarkupEditField())
- ->setKey('text')
- ->setLabel(pht('Document Body'))
- ->setDescription(pht('The body of text of the document.'))
- ->setConduitTypeDescription(pht('New document body.'))
- ->setValue($document_body)
- ->setIsRequired(true)
- ->setTransactionType(
- LegalpadDocumentTextTransaction::TRANSACTIONTYPE);
+ $fields[] =
+ id(new PhabricatorRemarkupEditField())
+ ->setKey('preamble')
+ ->setLabel(pht('Preamble'))
+ ->setDescription(pht('The preamble of the document.'))
+ ->setConduitTypeDescription(pht('New document preamble.'))
+ ->setValue($object->getPreamble())
+ ->setTransactionType(
+ LegalpadDocumentPreambleTransaction::TRANSACTIONTYPE);
+
+ $fields[] =
+ id(new PhabricatorRemarkupEditField())
+ ->setKey('text')
+ ->setLabel(pht('Document Body'))
+ ->setDescription(pht('The body of text of the document.'))
+ ->setConduitTypeDescription(pht('New document body.'))
+ ->setValue($document_body)
+ ->setIsRequired(true)
+ ->setTransactionType(
+ LegalpadDocumentTextTransaction::TRANSACTIONTYPE);
return $fields;

File Metadata

Mime Type
text/plain
Expires
Thu, Mar 27, 1:07 PM (4 d, 6 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7712685
Default Alt Text
D17875.id43001.diff (4 KB)

Event Timeline

Phabricator · Built by Phacility