Bug ReportBugs
ActivePublic

Properties

Description

This task was filed via the "New Bug Report" form.

Recent Activity

Today

wuzesheng added a comment to T12452: Can't edit task description after upgrading to the latest version.

And also, there's no edit form below this kind of task:

Mon, Mar 27, 5:39 AM · Bug Report
wuzesheng created T12452: Can't edit task description after upgrading to the latest version.
Mon, Mar 27, 5:37 AM · Bug Report

Yesterday

epriestley moved T12443: Applying fulltext limits first causes missing results from Backlog to v2 on the Search board.
Sun, Mar 26, 12:32 PM · Restricted Project, Search, Bug Report
epriestley closed T10161: Fulltext indexing produces invalid JSON documents in Elasticsearch as "Resolved".

Presuming this is either resolved by T9893/D17384 or no longer relevant. Follow up on T12450 or file a new task if you're still seeing issues.

Sun, Mar 26, 12:24 PM · Elasticsearch, Bug Report

Sat, Mar 25

epriestley closed T12449: Cannot set access control for creating new Maniphest forms as "Resolved".

I'm going to presume this is approximately resolved.

Sat, Mar 25, 1:09 AM · Bug Report

Fri, Mar 24

epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

(In the longer term, the permissions here could probably be cleaned up, but I think that's the cleanest fix without upstream changes.)

Fri, Mar 24, 10:53 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

You can edit the "Create Form" form and change "Visible To:" to "Members of Project: Doctorate in Form Theory". Then only members of that project will be able to create other forms or edit the form form, I believe.

Fri, Mar 24, 10:50 PM · Bug Report
alexmv added a comment to T12449: Cannot set access control for creating new Maniphest forms.

Enabling that form seems to have done the trick -- thanks! Since it got renamed at some point from "Create form," its existence was almost certainly confusing to some previous admin, which is what got us into this.

Fri, Mar 24, 10:46 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

I think there is also zero legitimate reason for the form-configuring-form to be editable and this entire workflow essentially represents a trap for the unwary.

Fri, Mar 24, 9:43 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

If you can't enable that form, you might need to go hunt it down in the database and manually enable it (or tell me what you hit and we'll fix it if it's a bug).

Fri, Mar 24, 9:38 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

Can you click that link, then "Enable Form"? Or do you get some kind of error?

Fri, Mar 24, 9:36 PM · Bug Report
leonardorccarvalho added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Works perfectly. Thanks a lot!

Fri, Mar 24, 9:36 PM · Bug Report
alexmv added a comment to T12449: Cannot set access control for creating new Maniphest forms.

I do now see that, but I'm unclear how to use its form to configure the form to configure the form to configure the form from.

Fri, Mar 24, 9:34 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

A MARVEL OF ACADEMIC PURITY, EDITENGINE PUSHES THE BOUNDARIES OF ADVANCED FORM THEORY TO NEW DEPTHS

Fri, Mar 24, 9:27 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

DO YOU HAVE ACCESS TO FORMS FOR CONFIGURING FORMS?!

Fri, Mar 24, 9:25 PM · Bug Report
alexmv created T12449: Cannot set access control for creating new Maniphest forms.
Fri, Mar 24, 9:22 PM · Bug Report
epriestley closed T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad) as "Resolved" by committing rP24b6c7d7186a: Allow users to resign if they have authority over any reviewer.
Fri, Mar 24, 8:14 PM · Bug Report
epriestley added a revision to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad): D17558: Allow users to resign if they have authority over any reviewer.
Fri, Mar 24, 8:07 PM · Bug Report
leonardorccarvalho added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Thanks for you help and I'm sorry the steps I provided are not enough to reproduce my problem. I will gather more information and post soon.

Fri, Mar 24, 7:22 PM · Bug Report
chad added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

4f9bbb63ae9b was tested on this server. very scientific.

Fri, Mar 24, 7:03 PM · Bug Report
chad added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

I am not able to reproduce the bug based on the steps you provided above. Do you have more steps or rules in place that I should be following?

Fri, Mar 24, 6:54 PM · Bug Report
epriestley added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Does your webserver error log have a more specific stack trace on the server? I can't reproduce this locally.

Fri, Mar 24, 6:53 PM · Bug Report
leonardorccarvalho created T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).
Fri, Mar 24, 6:41 PM · Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Yeah I can't seem to fix this ad hoc. My guess is it's never worked to allow you to skip the first tab. If I fix the error states, it then breaks existing tab panels. So we'd need to build a migration.

Fri, Mar 24, 6:40 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

I just plan to fix the bug. That's the hard part of a 2-person team with thousands of open issues. Pick and choose the battles.

Fri, Mar 24, 5:05 PM · Dashboards, Bug Report
epriestley closed T12422: Unable to navigate to "All Daemons" on Mobile as "Resolved" by committing rP0ffde484e5e7: Give Daemons a mobile menu.
Fri, Mar 24, 4:20 PM · Daemons, Mobile, Bug Report
epriestley added a revision to T12422: Unable to navigate to "All Daemons" on Mobile: D17554: Give Daemons a mobile menu.
Fri, Mar 24, 4:10 PM · Daemons, Mobile, Bug Report
epriestley merged task T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection into T11037: `bin/phd status` is misleading about local / remote daemons.
Fri, Mar 24, 3:07 PM · Restricted Project, Bug Report
epriestley added a comment to T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection.

Actually, this is a little different from what I've been seeing locally. I'm just going to merge this into T11037, which discusses some general improvements to phd status output.

Fri, Mar 24, 3:07 PM · Restricted Project, Bug Report
eliaspro added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Thinking about this again, what about making this hard-coded numbered list instead a list of draggable items like it's used in Menu Configuration?

Fri, Mar 24, 2:00 PM · Dashboards, Bug Report
chad claimed T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.
Fri, Mar 24, 1:36 PM · Dashboards, Bug Report
epriestley added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Based on like 10 seconds of looking at the code that's where I'd start. 💯

Fri, Mar 24, 1:35 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

amazing what sleep will accomplish

Fri, Mar 24, 1:30 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

I think i need to set the inputs with an actual key, not just [], so [$ii]

Fri, Mar 24, 1:29 PM · Dashboards, Bug Report
ioeric added a comment to T12118: Differential email does not contain reviewers/subscribers information anymore.

For me, I often need to click into the link to check if I am a reviewer or a subscriber. It would be nice if I could know who are the reviewers by looking at the email.

Fri, Mar 24, 10:36 AM · Restricted Project, Differential, Bug Report
epriestley claimed T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

We should still be able to handle this case, but I can take a look at what's happening since this is a nonstandard element and the fix is probably easy once I understand what's going on with the logic.

Fri, Mar 24, 10:30 AM · Dashboards, Bug Report
epriestley added a comment to T12118: Differential email does not contain reviewers/subscribers information anymore.

See also T5853 (distinguish between "blocking" and "reviewing" in email) and T10574 (provide more guidance around the pathway toward a revision becoming accepted).

Fri, Mar 24, 10:25 AM · Restricted Project, Differential, Bug Report
chad updated subscribers of T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

@epriestley as far as I can tell, empty tokenizers do not POST an input to evaluate.

Fri, Mar 24, 3:08 AM · Dashboards, Bug Report
neilfitz added a project to T12118: Differential email does not contain reviewers/subscribers information anymore: Restricted Project.

Our users are also missing this, since they can't tell whether they're a blocking/lone reviewer from the e-mail.

Fri, Mar 24, 12:14 AM · Restricted Project, Differential, Bug Report

Thu, Mar 23

chad closed T12445: Logical Error in diffusion_existing.diviner as "Resolved" by committing rP4f2bca58fcea: Fix typo in diviner user guide / diffusion.
Thu, Mar 23, 7:43 PM · Bug Report
chad added a revision to T12445: Logical Error in diffusion_existing.diviner: D17546: Fix typo in diviner user guide / diffusion.
Thu, Mar 23, 7:23 PM · Bug Report
friesenschrauber created T12445: Logical Error in diffusion_existing.diviner.
Thu, Mar 23, 7:10 PM · Bug Report
epriestley closed T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation? as "Resolved".

Great! Thanks for the report, and for your help confirming the fix. Let us know if you run into anything else.

Thu, Mar 23, 6:12 PM · Bug Report
deathsyn added a comment to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?.

Checked out that commit and gave it a run. Profile images all generated without errors and I can now reach the front pages of Maniphest and People successfully.

Thu, Mar 23, 6:05 PM · Bug Report
epriestley added a comment to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?.

Thanks! That's consistent with 32-bit and I suspect the & 0x7FFFFFFF part of rP9326b4d131ce will fix it.

Thu, Mar 23, 6:05 PM · Bug Report
deathsyn added a comment to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?.

@deathsyn Can you show me the output of this command on your system to help confirm that there's a 32-bit issue at play here?

$ php -r 'echo PHP_INT_MAX."\n";'

The other bugs I'm more certain are actual bugs, but I'm not totally sure about that one.

Thu, Mar 23, 6:04 PM · Bug Report
epriestley added a comment to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?.

(Alternatively, upgrade to rP9326b4d131ce and run the bin/people ... command again and see if that just fixes it.)

Thu, Mar 23, 5:56 PM · Bug Report
epriestley added a commit to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?: rP9326b4d131ce: Fix some range issues and 32-bit issues with avatar generation.
Thu, Mar 23, 5:51 PM · Bug Report
epriestley added a comment to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?.

@deathsyn Can you show me the output of this command on your system to help confirm that there's a 32-bit issue at play here?

Thu, Mar 23, 5:49 PM · Bug Report
epriestley added a revision to T12444: Can't reach Maniphest or People front pages on stable upgrade because of Avatar creation?: D17543: Fix some range issues and 32-bit issues with avatar generation.
Thu, Mar 23, 5:48 PM · Bug Report