Page MenuHomePhabricator
Files F19019846

D21210.id.diff
No OneTemporary
Actions

D21210.id.diff
View Options

diff --git a/src/applications/differential/parser/DifferentialChangesetParser.php b/src/applications/differential/parser/DifferentialChangesetParser.php
--- a/src/applications/differential/parser/DifferentialChangesetParser.php
+++ b/src/applications/differential/parser/DifferentialChangesetParser.php
@@ -592,6 +592,17 @@
}
private function tryCacheStuff() {
+ $changeset = $this->getChangeset();
+ if (!$changeset->hasSourceTextBody()) {
+
+ // TODO: This isn't really correct (the change is not "generated"), the
+ // intent is just to not render a text body for Subversion directory
+ // changes, etc.
+ $this->markGenerated();
+
+ return;
+ }
+
$viewstate = $this->getViewState();
$skip_cache = false;
@@ -610,19 +621,10 @@
$skip_cache = true;
}
- $changeset = $this->changeset;
-
- if ($changeset->getFileType() != DifferentialChangeType::FILE_TEXT &&
- $changeset->getFileType() != DifferentialChangeType::FILE_SYMLINK) {
-
- $this->markGenerated();
-
- } else {
- if ($skip_cache || !$this->loadCache()) {
- $this->process();
- if (!$skip_cache) {
- $this->saveCache();
- }
+ if ($skip_cache || !$this->loadCache()) {
+ $this->process();
+ if (!$skip_cache) {
+ $this->saveCache();
}
}
}
diff --git a/src/applications/differential/storage/DifferentialChangeset.php b/src/applications/differential/storage/DifferentialChangeset.php
--- a/src/applications/differential/storage/DifferentialChangeset.php
+++ b/src/applications/differential/storage/DifferentialChangeset.php
@@ -32,6 +32,8 @@
private $hasNewState;
private $oldStateMetadata;
private $newStateMetadata;
+ private $oldFileType;
+ private $newFileType;
const TABLE_CACHE = 'differential_changeset_parse_cache';
@@ -550,17 +552,20 @@
$left_metadata = $left->getNewStateMetadata();
$left_state = $left->hasNewState();
$shared_metadata = $left->getMetadata();
+ $left_type = $left->getNewFileType();
if ($right) {
$right_data = $right->makeNewFile();
$right_properties = $right->getNewProperties();
$right_metadata = $right->getNewStateMetadata();
$right_state = $right->hasNewState();
$shared_metadata = $right->getMetadata();
+ $right_type = $right->getNewFileType();
} else {
$right_data = $left->makeOldFile();
$right_properties = $left->getOldProperties();
$right_metadata = $left->getOldStateMetadata();
$right_state = $left->hasOldState();
+ $right_type = $left->getOldFileType();
}
$engine = new PhabricatorDifferenceEngine();
@@ -576,7 +581,6 @@
->setFilename($right->getFilename());
// TODO: Change type?
- // TODO: File type?
// TODO: Away paths?
// TODO: View state key?
@@ -589,7 +593,9 @@
->setOldStateMetadata($left_metadata)
->setNewStateMetadata($right_metadata)
->setHasOldState($left_state)
- ->setHasNewState($right_state);
+ ->setHasNewState($right_state)
+ ->setOldFileType($left_type)
+ ->setNewFileType($right_type);
// NOTE: Some metadata is not stored statefully, like the "generated"
// flag. For now, use the rightmost "new state" metadata to fill in these
@@ -602,6 +608,45 @@
return $comparison;
}
+
+ public function setNewFileType($new_file_type) {
+ $this->newFileType = $new_file_type;
+ return $this;
+ }
+
+ public function getNewFileType() {
+ if ($this->newFileType !== null) {
+ return $this->newFileType;
+ }
+
+ return $this->getFiletype();
+ }
+
+ public function setOldFileType($old_file_type) {
+ $this->oldFileType = $old_file_type;
+ return $this;
+ }
+
+ public function getOldFileType() {
+ if ($this->oldFileType !== null) {
+ return $this->oldFileType;
+ }
+
+ return $this->getFileType();
+ }
+
+ public function hasSourceTextBody() {
+ $type_map = array(
+ DifferentialChangeType::FILE_TEXT => true,
+ DifferentialChangeType::FILE_SYMLINK => true,
+ );
+
+ $old_body = isset($type_map[$this->getOldFileType()]);
+ $new_body = isset($type_map[$this->getNewFileType()]);
+
+ return ($old_body || $new_body);
+ }
+
public function getNewStateMetadata() {
return $this->getMetadataWithPrefix('new:');
}

File Metadata

Mime Type
text/plain
Expires
Mon, Nov 24, 4:04 PM (5 d, 9 m ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
8527865
Default Alt Text
D21210.id.diff (4 KB)

Event Timeline

Phabricator · Built by Phacility