Page MenuHomePhabricator
Feed Advanced Search

Sep 22 2016

michaeloa added a comment to T11683: Attempt to connect to phabricator@p:127.0.0.1 failed with error #1040: Too many connections..

When rebooting the sql server, we get an error on:

Sep 22 2016, 9:37 AM · Bug Report
michaeloa created T11683: Attempt to connect to phabricator@p:127.0.0.1 failed with error #1040: Too many connections..
Sep 22 2016, 9:32 AM · Bug Report
mna created T11682: Numbered lists are overzealous.
Sep 22 2016, 9:11 AM · Bug Report
• zhengdifei added a comment to T11681: emulator test.

搭建 emulator 的 code review

Sep 22 2016, 6:06 AM · Bug Report
• zhengdifei created T11681: emulator test.
Sep 22 2016, 6:05 AM · Bug Report
avivey updated the task description for T10944: Subproject is not archived when parent project gets archived.
Sep 22 2016, 5:52 AM · Projects, Bug Report
epriestley added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

This should now just work without any changes at HEAD of master. Let us know if you're still seeing issues after updating.

Sep 22 2016, 1:24 AM · Custom Fields, Maniphest, Bug Report
epriestley added a comment to T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).

I think this should work better at HEAD of master now -- let us know if you're still seeing issues after updating. Thanks for the report!

Sep 22 2016, 1:23 AM · Maniphest, Bug Report
epriestley closed T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped) as Resolved by committing Restricted Diffusion Commit.
Sep 22 2016, 1:22 AM · Maniphest, Bug Report

Sep 21 2016

epriestley closed T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form as Resolved by committing Restricted Diffusion Commit.
Sep 21 2016, 11:39 PM · Custom Fields, Maniphest, Bug Report
epriestley claimed T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.
Sep 21 2016, 10:07 PM · Custom Fields, Maniphest, Bug Report
epriestley claimed T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).
Sep 21 2016, 10:06 PM · Maniphest, Bug Report
epriestley closed T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks as Resolved.

Presuming resolved by D16584. Thanks!

Sep 21 2016, 10:06 PM · Bug Report
bzerr added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

Submitted {D16584}

Sep 21 2016, 9:52 PM · Bug Report
epriestley added a revision to T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped): Restricted Differential Revision.
Sep 21 2016, 9:45 PM · Maniphest, Bug Report
epriestley added a revision to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form: Restricted Differential Revision.
Sep 21 2016, 9:35 PM · Custom Fields, Maniphest, Bug Report
epriestley added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

There's a bunch of other Herald stuff we already do for this field so I'm just undoing the change from D14766, at least until someone complains that this field type isn't using the search logic they expect.

Sep 21 2016, 9:34 PM · Custom Fields, Maniphest, Bug Report
alexmv closed T11248: ArcanistLinter's addLintMessage incorrectly normalizes path on Windows as Resolved by committing Restricted Diffusion Commit.
Sep 21 2016, 9:29 PM · Windows, Bug Report
epriestley added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

Yeah, go ahead and submit a patch. Thanks!

Sep 21 2016, 9:28 PM · Bug Report
chad updated subscribers of T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

Minimum viable patch sounds good to me, but @epriestley handles all contribs. Not sure what he prefers here. Feel free to patch locally first and verify.

Sep 21 2016, 9:10 PM · Bug Report
hach-que added a comment to T11679: Phragment has invalid (non-UTF8) title glyph.

Pretty sure it was copy pasted from something else at the time; I don't believe I would have actually picked a glyph.

Sep 21 2016, 8:55 PM · Contributor Onboarding, Phragment, Bug Report
bzerr added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

So I've figured out the underlying cause. Outlook is sending attachments without a Content-Disposition header. The current version of mimemailparser included with Phabricator completely ignores attachments without that header.

Sep 21 2016, 8:36 PM · Bug Report
alexmv added a comment to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

This is still logged as an EXCEPTION. Is DELETE FROM phabricator_repository.edge WHERE type = 39 a reasonable step to take, to prevent administrators unfamiliar with the non-fatal-ness of this from being worried?

Sep 21 2016, 7:07 PM · Bug Report
epriestley triaged T11679: Phragment has invalid (non-UTF8) title glyph as Wishlist priority.
Sep 21 2016, 5:57 PM · Contributor Onboarding, Phragment, Bug Report
epriestley renamed T11679: Phragment has invalid (non-UTF8) title glyph from Phragment on secure.phabricator.com crashes to Phragment has invalid (non-UTF8) title glyph.
Sep 21 2016, 5:57 PM · Contributor Onboarding, Phragment, Bug Report
epriestley moved T11679: Phragment has invalid (non-UTF8) title glyph from Backlog to Ready on the Contributor Onboarding board.
Sep 21 2016, 5:56 PM · Contributor Onboarding, Phragment, Bug Report
epriestley added a project to T11679: Phragment has invalid (non-UTF8) title glyph: Contributor Onboarding.

To fix:

Sep 21 2016, 5:56 PM · Contributor Onboarding, Phragment, Bug Report
epriestley updated subscribers of T11679: Phragment has invalid (non-UTF8) title glyph.

We don't normally accept bug reports on prototype applications (see Prototype) but this is a one-line fix.

Sep 21 2016, 5:48 PM · Contributor Onboarding, Phragment, Bug Report
avivey added a project to T11679: Phragment has invalid (non-UTF8) title glyph: Phragment.
Sep 21 2016, 5:32 PM · Contributor Onboarding, Phragment, Bug Report
mna created T11679: Phragment has invalid (non-UTF8) title glyph.
Sep 21 2016, 5:29 PM · Contributor Onboarding, Phragment, Bug Report
bzerr added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

Makes sense, these are old versions of software. I will test with more recent versions of both Windows and Outlook to see if the issue persists.

Sep 21 2016, 5:23 PM · Bug Report
epriestley moved T11667: Default global syntax.filemap sets default filetypes to JavaScript for JSON configurations from Backlog to Ready on the Contributor Onboarding board.
Sep 21 2016, 5:01 PM · Contributor Onboarding, Bug Report
epriestley added a project to T11667: Default global syntax.filemap sets default filetypes to JavaScript for JSON configurations: Contributor Onboarding.

To fix this:

Sep 21 2016, 5:01 PM · Contributor Onboarding, Bug Report
chad added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

Or I should say, the amount of time that would be needed to get to Windows 7 issues, would negate the need to support it. We're two developers and any bug is competing with thousands of others.

Sep 21 2016, 4:55 PM · Bug Report
chad added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

We have no plans to support Windows 7. Do you have issues with more modern versions of Outlook?

Sep 21 2016, 4:54 PM · Bug Report
ReynoldsCahoon added a comment to T10612: Editing a panel causes it to duplicate in dashboard.

Having the same issue. Seems to only occur on certain panels. Adding the panel in question to a column now creates a duplicate in the opposite column. Removing the panel from one column removes it from both.

Sep 21 2016, 4:52 PM · Dashboards (v2), Bug Report
bzerr added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

With the .eml files, you could change the From: and To: headers and pipe directly into sendmail or similar MTA.
If that doesn't work, I would be happy to provide a VM instance to assist in reproduction.

Sep 21 2016, 4:47 PM · Bug Report
chad added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

How can we reproduce this locally? Do you have a specific version of Outlook / OS you are using?

Sep 21 2016, 4:08 PM · Bug Report
bzerr created T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.
Sep 21 2016, 3:36 PM · Bug Report
20after4 added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

We use a text field for a 'reference number' which refers to tasks imported from bugzilla reference=bz123, or RT with reference=rt456

Sep 21 2016, 3:14 PM · Custom Fields, Maniphest, Bug Report
epriestley added a comment to T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).

I think this is the CSS that tries to cut off the titles interacting weirdly with other table CSS.

Sep 21 2016, 2:37 PM · Maniphest, Bug Report
epriestley added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

I believed this was semi-intentional, although the downstream report suggests it worked at one point, so maybe I just made a mistake.

Sep 21 2016, 2:23 PM · Custom Fields, Maniphest, Bug Report
chad closed T11674: Phabricator mobile UI does not show dashboard that is installed as Invalid.

Click "home". to see your installed dashboard. This is the expected UI.

Sep 21 2016, 12:19 PM · Bug Report
aklapper created T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).
Sep 21 2016, 10:32 AM · Maniphest, Bug Report
aklapper created T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.
Sep 21 2016, 10:08 AM · Custom Fields, Maniphest, Bug Report
• KalleKyllonen created T11674: Phabricator mobile UI does not show dashboard that is installed .
Sep 21 2016, 9:48 AM · Bug Report

Sep 20 2016

alexmv added a revision to T11248: ArcanistLinter's addLintMessage incorrectly normalizes path on Windows: Restricted Differential Revision.
Sep 20 2016, 10:05 PM · Windows, Bug Report
chad merged T11673: Timeline context menu doesn't disappear when taking action into T10400: Dropdown menus can overlap.
Sep 20 2016, 9:51 PM · PHUI, Bug Report
chad merged task T11673: Timeline context menu doesn't disappear when taking action into T10400: Dropdown menus can overlap.
Sep 20 2016, 9:51 PM · Bug Report
cspeckmim created T11673: Timeline context menu doesn't disappear when taking action.
Sep 20 2016, 6:27 PM · Bug Report
sascha-egerer added a comment to T10725: Can't access binary files of differential revision.

I have that problem again. There is no relation between the file and the Diff. Don't know how that can happen...

Sep 20 2016, 4:06 PM · Restricted Project, Differential, Bug Report
epriestley merged task T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception into T8937: Unclear exception when patching non-existant revision.
Sep 20 2016, 1:33 PM · Arcanist, Bug Report
cspeckmim updated the task description for T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception.
Sep 20 2016, 1:26 PM · Arcanist, Bug Report
cspeckmim created T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception.
Sep 20 2016, 1:26 PM · Arcanist, Bug Report
epriestley added a comment to T11668: Herald editing view show unknown error .

Is this reproducible using fields available in the upstream? How can I reproduce this issue against a clean, vanilla, up-to-date copy of Phabricator with no custom code?

Sep 20 2016, 9:50 AM · Bug Report (Needs Information)
• KalleKyllonen added a comment to T11668: Herald editing view show unknown error .

We have defined that one. It is like our teams with the company. When new bug is created for example maps team, herald automatically sets the right person/team for the bug.

Sep 20 2016, 9:46 AM · Bug Report (Needs Information)
epriestley added a comment to T11668: Herald editing view show unknown error .

Where is "Platform--MUST-->" coming from? That isn't a field we provide in the upstream.

Sep 20 2016, 9:39 AM · Bug Report (Needs Information)
• KalleKyllonen created T11669: Dashboard drag and drop a test in patience.
Sep 20 2016, 9:25 AM · Restricted Project, Dashboards, Bug Report
• KalleKyllonen added a comment to T11668: Herald editing view show unknown error .

phabricator 51f8ec4487be02e290b1182a7cd5b4069fe7d49f (Tue, Sep 20) arcanist 9e82ef979e8148c43b9b8439025d505b1219e213 (Thu, Aug 25) phutil fa6519f4cea1c059bd381c20971cac413cc4760e (Thu, Sep 15)

Sep 20 2016, 9:12 AM · Bug Report (Needs Information)
• KalleKyllonen created T11668: Herald editing view show unknown error .
Sep 20 2016, 9:10 AM · Bug Report (Needs Information)
• KalleKyllonen added a watcher for Bug Report: • KalleKyllonen.
Sep 20 2016, 9:06 AM
cinek created T11667: Default global syntax.filemap sets default filetypes to JavaScript for JSON configurations.
Sep 20 2016, 7:31 AM · Contributor Onboarding, Bug Report
cspeckmim renamed T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured from Differential comment panel haunter does not allow scrolling to bottom of differential revision to Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.
Sep 20 2016, 4:04 AM · Differential, Bug Report
cspeckmim added a comment to T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.

Ok I have observed this now on this install, however only in the second haunt mode Z+Z:
https://secure.phabricator.com/D15067

Sep 20 2016, 4:01 AM · Differential, Bug Report
swisspol updated subscribers of T11666: Checkout is failing from Phacility.

@epriestley Did you see the AphrontQueryException: #1364: Field 'messageCount' doesn't have a default value problem too?

Sep 20 2016, 1:57 AM · Bug Report
swisspol added a comment to T11666: Checkout is failing from Phacility.

Hummm I don't know if it was a transient error or I just happened to magically fix it by force pushing to the repo, but git pulls now work again on my Mac and in CI.

Sep 20 2016, 1:55 AM · Bug Report
epriestley merged task T11666: Checkout is failing from Phacility into T11665: repo002.phacility.net is heavily loaded.
Sep 20 2016, 1:51 AM · Bug Report
swisspol added a comment to T11666: Checkout is failing from Phacility.

Pushing appears to work (as in the commits appear in Diffusion) but errors are printed:

$ git push
# Push received by "web.phacility.net", forwarding to cluster host.
# Waiting up to 120 second(s) for a cluster write lock...
# Acquired write lock immediately.
# Waiting up to 120 second(s) for a cluster read lock on "repo002.phacility.net"...
# Acquired read lock immediately.
# Device "repo002.phacility.net" is already a cluster leader and does not need to be synchronized.
# Ready to receive on cluster host "repo002.phacility.net".
Counting objects: 11, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 1.90 KiB | 0 bytes/s, done.
Total 11 (delta 10), reused 0 (delta 0)
# Released cluster write lock.
PHP Fatal error:  Uncaught exception 'FilesystemException' with message 'Requested path '/core/log/hyper' is not writable.' in /core/lib/libphutil/src/filesystem/Filesystem.php:1107
Stack trace:
#0 /core/lib/libphutil/src/filesystem/Filesystem.php(650): Filesystem::assertWritable('/core/log/hyper')
#1 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(183): Filesystem::createDirectory('/core//log/hype...', 493, true)
#2 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(155): PhutilDeferredLog->write()
#3 [internal function]: PhutilDeferredLog->__destruct()
#4 {main}
  thrown in /core/lib/libphutil/src/filesystem/Filesystem.php on line 1107
To ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
   d77b984..6eb6700  master -> master
error: failed to push some refs to 'ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git'
Sep 20 2016, 1:50 AM · Bug Report
swisspol created T11666: Checkout is failing from Phacility.
Sep 20 2016, 1:48 AM · Bug Report

Sep 19 2016

cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Ohhh nice, thank you I missed that
=C124

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

And you can technically:

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Oh, right. That works, but I think isn't documented:

Sep 19 2016, 9:31 PM · Documentation, Remarkup
cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

It's an off-case for sure. We have a ton of data identifiers which all end with an equals-sign and a couple letters/numbers, which we abbreviate when referring to the datasets. I was actually trying to render the equals sign as part of the nav sequence and not just C124.

Sep 19 2016, 9:30 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

We could conceivably special-case this since setting an empty attribute to a value isn't meaningful, I suppose.

Sep 19 2016, 9:25 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
{nav name=C124}
Sep 19 2016, 9:24 PM · Documentation, Remarkup
cspeckmim created T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
Sep 19 2016, 9:04 PM · Documentation, Remarkup
avivey updated the task description for T11660: Diff creation fails when submitting too long file paths.
Sep 19 2016, 5:53 PM · Differential, Bug Report
benjamin.cohen-solal added a project to T11660: Diff creation fails when submitting too long file paths: Differential.
Sep 19 2016, 2:52 PM · Differential, Bug Report
benjamin.cohen-solal created T11660: Diff creation fails when submitting too long file paths.
Sep 19 2016, 2:52 PM · Differential, Bug Report
cspeckmim created T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.
Sep 19 2016, 1:41 PM · Differential, Bug Report

Sep 18 2016

chad closed T11654: Keyboard shortcuts (?) show as white on gray (unreadable) as Resolved by committing Restricted Diffusion Commit.
Sep 18 2016, 10:35 PM · Bug Report
chad added a revision to T11654: Keyboard shortcuts (?) show as white on gray (unreadable): Restricted Differential Revision.
Sep 18 2016, 10:18 PM · Bug Report
chad closed T11656: Phabricator as Spite.
Sep 18 2016, 9:49 PM · Bug Report
chad closed T11655: Phabricator as Spite.
Sep 18 2016, 9:48 PM · Bug Report
• lpetov.ad36 created T11656: Phabricator.
Sep 18 2016, 9:04 PM · Bug Report
• lpetov.ad36 created T11655: Phabricator.
Sep 18 2016, 9:03 PM · Bug Report
carlokok created T11654: Keyboard shortcuts (?) show as white on gray (unreadable).
Sep 18 2016, 6:09 PM · Bug Report

Sep 17 2016

d.maznekov merged task T11653: Take over (forcibly) Permisions of Project into T10255: Edit permissions for Project (before and after some update of Phabricator).
Sep 17 2016, 10:04 AM · Bug Report
d.maznekov merged T11653: Take over (forcibly) Permisions of Project into T10255: Edit permissions for Project (before and after some update of Phabricator).
Sep 17 2016, 10:04 AM · Bug Report
d.maznekov added a comment to T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:59 AM · Bug Report
d.maznekov merged task T10255: Edit permissions for Project (before and after some update of Phabricator) into T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:59 AM · Bug Report
d.maznekov merged T10255: Edit permissions for Project (before and after some update of Phabricator) into T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:58 AM · Bug Report
d.maznekov renamed T11653: Take over (forcibly) Permisions of Project from Permisions of Project to Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:53 AM · Bug Report
d.maznekov created T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:52 AM · Bug Report

Sep 16 2016

bizrad6 added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Awesome! Thanks for the quick fix @epriestley!

Sep 16 2016, 9:37 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

I think the first part of this should be fixed in HEAD now, thanks for the report!

Sep 16 2016, 9:07 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

The "multiple packages" issue may have been a symptom of the dominion/archive problem, here's a case I hit while testing that made it look like there was a second issue:

Sep 16 2016, 9:00 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

D16564 might fix this but I need to actually test it.

Sep 16 2016, 7:10 PM · Restricted Project, Bug Report
epriestley added a revision to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived: Restricted Differential Revision.
Sep 16 2016, 7:09 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Also, what is the expected behavior if two Owners packages point to the same path?

Sep 16 2016, 7:02 PM · Restricted Project, Bug Report