Page MenuHomePhabricator
Feed Advanced Search

Apr 1 2016

epriestley closed T10704: Comments on images in Differential (Right Side) cause JS error as Resolved by committing Restricted Diffusion Commit.
Apr 1 2016, 7:54 PM · Differential, Bug Report
epriestley added a revision to T10704: Comments on images in Differential (Right Side) cause JS error: Restricted Differential Revision.
Apr 1 2016, 7:53 PM · Differential, Bug Report
epriestley added a comment to T10704: Comments on images in Differential (Right Side) cause JS error.

I think a workaround is to swap the particular changeset into unified mode (View Options → View Unified) and then interact with the comment, but the fix seems trivial and straightforward.

Apr 1 2016, 7:51 PM · Differential, Bug Report
chad renamed T10704: Comments on images in Differential (Right Side) cause JS error from Comments on images in Differential (Right Side) to Comments on images in Differential (Right Side) cause JS error.
Apr 1 2016, 7:50 PM · Differential, Bug Report
chad renamed T10704: Comments on images in Differential (Right Side) cause JS error from comments on images in Differential broken to Comments on images in Differential (Right Side).
Apr 1 2016, 7:50 PM · Differential, Bug Report
epriestley claimed T10704: Comments on images in Differential (Right Side) cause JS error.
Apr 1 2016, 7:48 PM · Differential, Bug Report
epriestley added a comment to T10704: Comments on images in Differential (Right Side) cause JS error.

Thanks, I can reproduce this as well (e.g. D15565).

Apr 1 2016, 7:48 PM · Differential, Bug Report
allixsenos created T10704: Comments on images in Differential (Right Side) cause JS error.
Apr 1 2016, 7:40 PM · Differential, Bug Report

Mar 31 2016

avivey added a comment to T10687: jx-tooltip does whatever it feels like.

D14879 made hovercards smart about these sort of things (But hovercard don't have a little arrow thingy).

Mar 31 2016, 1:34 AM · PHUI, Bug Report

Mar 30 2016

avivey added a project to T10692: misleading arc lint / autofix docs: Documentation.
Mar 30 2016, 1:50 AM · Documentation, Bug Report

Mar 29 2016

allixsenos added a comment to T10692: misleading arc lint / autofix docs.

sorry, forgot to say it's this document https://secure.phabricator.com/book/phabricator/article/arcanist_lint/

Mar 29 2016, 11:39 PM · Documentation, Bug Report
allixsenos created T10692: misleading arc lint / autofix docs.
Mar 29 2016, 11:38 PM · Documentation, Bug Report
avivey added a comment to T10687: jx-tooltip does whatever it feels like.

I always see it like this (Chrome/Windows); I've dug a little in the algorithm, and I think it's using location relative to the page for the calculation, which is why the first (top-most) tooltip acts differently than all the rest.

Mar 29 2016, 6:22 PM · PHUI, Bug Report
avivey updated the task description for T10687: jx-tooltip does whatever it feels like.
Mar 29 2016, 6:16 PM · PHUI, Bug Report
lpriestley closed T10688: Badge translation error as Resolved by committing Restricted Diffusion Commit.
Mar 29 2016, 5:56 PM · Badges, Bug Report
lpriestley added a revision to T10688: Badge translation error: Restricted Differential Revision.
Mar 29 2016, 5:38 PM · Badges, Bug Report
epriestley added a comment to T10688: Badge translation error.

I think these steps should reproduce the issue:

Mar 29 2016, 5:22 PM · Badges, Bug Report
lpriestley added a comment to T10688: Badge translation error.

How do I reproduce this error?

Mar 29 2016, 5:20 PM · Badges, Bug Report
epriestley assigned T10688: Badge translation error to lpriestley.

D15545 should fix the infrastructure part of this, and make this error more clear by returning an explicit diagnostic error rather than a mostly-working string and a log.

Mar 29 2016, 12:33 PM · Badges, Bug Report
epriestley added a revision to T10688: Badge translation error: Restricted Differential Revision.
Mar 29 2016, 12:31 PM · Badges, Bug Report
epriestley added a comment to T10688: Badge translation error.

Oh, I believe the actual issue is that the translations are variant on the wrong parameter. I think this is the correct fix (adding an extra array()):

Mar 29 2016, 12:10 PM · Badges, Bug Report
epriestley added a comment to T10688: Badge translation error.

D15544 (changing %s to %d) shouldn't be the fix for this -- %s is correct, because we want to render "1,234,567" if the number is large:

Mar 29 2016, 12:08 PM · Badges, Bug Report
cspeckmim added a comment to T10687: jx-tooltip does whatever it feels like.

Are you always seeing this render in a bad location or is it only sometimes? I regularly see the first rendered hovercard located way off to the left but it's only the first hovercard that tries to render -- and it's usually only the first hovercard after starting up Firefox. I haven't found reproducible steps so I haven't bothered to report it yet, but I wonder if it's related to this. I primarily use (the most recent) Firefox so it may be just browser. Using Cmd/Ctrl+Refresh doesn't cause it to affect again - manually restarting Firefox is also not a surefire way to reproduce.

Mar 29 2016, 6:36 AM · PHUI, Bug Report
chad added a revision to T10688: Badge translation error: Restricted Differential Revision.
Mar 29 2016, 3:04 AM · Badges, Bug Report
chad created T10688: Badge translation error.
Mar 29 2016, 2:37 AM · Badges, Bug Report
avivey created T10687: jx-tooltip does whatever it feels like.
Mar 29 2016, 12:44 AM · PHUI, Bug Report

Mar 28 2016

eadler moved T10225: arc patch --nobranch leaves submodules in the wrong state from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Mar 28 2016, 8:29 PM · Haskell.org, Restricted Project, Arcanist, Bug Report
eadler moved T10682: Applications have no profile photos when they author transactions from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Mar 28 2016, 8:29 PM · Restricted Project, Harbormaster, Bug Report
eadler added a project to T10225: arc patch --nobranch leaves submodules in the wrong state: Restricted Project.
Mar 28 2016, 8:23 PM · Haskell.org, Restricted Project, Arcanist, Bug Report
eadler added a project to T10682: Applications have no profile photos when they author transactions: Restricted Project.
Mar 28 2016, 8:19 PM · Restricted Project, Harbormaster, Bug Report
epriestley updated subscribers of T10234: When using a PhabricatorRemarkupCustomInlineRule with caching disabled, custom rules may end up with incorrect engines.

D15535 should completely fix this in all cases

Mar 28 2016, 5:36 PM · Remarkup, Bug Report
epriestley added a revision to T10234: When using a PhabricatorRemarkupCustomInlineRule with caching disabled, custom rules may end up with incorrect engines: Restricted Differential Revision.
Mar 28 2016, 5:24 PM · Remarkup, Bug Report
cspeckmim added a comment to T10686: Remarkup: Unable to use italics on a quoted header line.

Evidence for the trial (when the fix goes live the description will render correctly)

Mar 28 2016, 4:33 PM · Remarkup, Bug Report
cspeckmim added a comment to T10686: Remarkup: Unable to use italics on a quoted header line.

It was another cspeckmim

Mar 28 2016, 4:23 PM · Remarkup, Bug Report
epriestley closed T10686: Remarkup: Unable to use italics on a quoted header line as Resolved by committing Restricted Diffusion Commit.
Mar 28 2016, 4:12 PM · Remarkup, Bug Report
epriestley added a revision to T10686: Remarkup: Unable to use italics on a quoted header line: Restricted Differential Revision.
Mar 28 2016, 4:11 PM · Remarkup, Bug Report
epriestley claimed T10686: Remarkup: Unable to use italics on a quoted header line.

u wot m8? havin' a go? havin' a little laff?

Mar 28 2016, 4:09 PM · Remarkup, Bug Report
cspeckmim updated subscribers of T10686: Remarkup: Unable to use italics on a quoted header line.
Mar 28 2016, 4:06 PM · Remarkup, Bug Report
cspeckmim created T10686: Remarkup: Unable to use italics on a quoted header line.
Mar 28 2016, 4:05 PM · Remarkup, Bug Report

Mar 27 2016

epriestley added a parent task for T10683: New Conpherence participant makes the conversation unread but does not update subtitle: T10364: Conpherence v4 (UI/UX).
Mar 27 2016, 7:34 PM · Conpherence (v4), Bug Report
Krenair created T10683: New Conpherence participant makes the conversation unread but does not update subtitle.
Mar 27 2016, 7:32 PM · Conpherence (v4), Bug Report
swisspol added a comment to T10682: Applications have no profile photos when they author transactions.

A generic Phab icon would be fine for a first pass at improving this IMO.

Mar 27 2016, 7:02 PM · Restricted Project, Harbormaster, Bug Report
epriestley added a comment to T10682: Applications have no profile photos when they author transactions.

At a minimum, Herald, Harbormaster, Nuance, Phortune and Differential can author stories today, although not all of these stories currently render with photos. We should probably assume any application may author stories.

Mar 27 2016, 7:00 PM · Restricted Project, Harbormaster, Bug Report
epriestley renamed T10682: Applications have no profile photos when they author transactions from Harbormaster has no "user photo" in feed to Applications have no profile photos when they author transactions.
Mar 27 2016, 6:58 PM · Restricted Project, Harbormaster, Bug Report
chad added a project to T10682: Applications have no profile photos when they author transactions: Harbormaster.
Mar 27 2016, 6:11 PM · Restricted Project, Harbormaster, Bug Report
epriestley added a comment to T10682: Applications have no profile photos when they author transactions.

See also vaguely-related T8952.

Mar 27 2016, 5:33 PM · Restricted Project, Harbormaster, Bug Report
swisspol updated the task description for T10682: Applications have no profile photos when they author transactions.
Mar 27 2016, 5:05 PM · Restricted Project, Harbormaster, Bug Report
swisspol created T10682: Applications have no profile photos when they author transactions.
Mar 27 2016, 5:04 PM · Restricted Project, Harbormaster, Bug Report
epriestley merged task T10680: If Audit app is uninstalled, you can still add comments to a commit and "raise concern" but it fails with 404 into T6630: Uninstalling Audit should either be impossible or interact more cleanly with Diffusion.
Mar 27 2016, 5:01 PM · Bug Report
swisspol created T10680: If Audit app is uninstalled, you can still add comments to a commit and "raise concern" but it fails with 404.
Mar 27 2016, 4:57 PM · Bug Report

Mar 26 2016

epriestley closed T10302: Workflow.js is picking up 7px from somewhere as Resolved by committing Restricted Diffusion Commit.
Mar 26 2016, 7:29 PM · PHUI, Bug Report
epriestley added a revision to T10302: Workflow.js is picking up 7px from somewhere: Restricted Differential Revision.
Mar 26 2016, 7:26 PM · PHUI, Bug Report
epriestley closed T10356: Unhandled Exception ("FilesystemException") after adding Github repo as Invalid.

This report does not include reproduction steps. Bug reports must include complete, self-contained reproduction steps to be accepted by the upstream. See Providing Reproduction Steps for help.

Mar 26 2016, 7:15 PM · Diffusion, Bug Report
chad closed T10676: Shut out from a project as Resolved.

I think you are missing some steps, like that you need to set the edit policy to something fairly restrictive. @richardvanvelzen is correct though, you can unlock this from the command line and archive the project. You can also use bin/remove to permanently nuke objects, but it's not recommended.

Mar 26 2016, 2:57 PM · Projects, Bug Report
richardvanvelzen added a comment to T10676: Shut out from a project.

You can probably use bin/policy unlock to open up the policy.

Mar 26 2016, 12:35 PM · Projects, Bug Report
spidfire created T10676: Shut out from a project.
Mar 26 2016, 12:01 PM · Projects, Bug Report
ssangervasi created T10675: Pytest engine should not execute "coverage" if --no-coverage flag is provided. .
Mar 26 2016, 12:07 AM · Unit, Arcanist, Bug Report

Mar 25 2016

epriestley closed T10674: Latest HEAD broke viewing unit test results in Harbormaster as Resolved by committing Restricted Diffusion Commit.
Mar 25 2016, 10:49 PM · Harbormaster, Bug Report
epriestley added a revision to T10674: Latest HEAD broke viewing unit test results in Harbormaster: Restricted Differential Revision.
Mar 25 2016, 10:46 PM · Harbormaster, Bug Report
thoughtpolice created T10674: Latest HEAD broke viewing unit test results in Harbormaster.
Mar 25 2016, 10:27 PM · Harbormaster, Bug Report
chad added a project to T10661: Can't create milestone: Projects (v3).
Mar 25 2016, 7:11 PM · Projects (v3), Bug Report
epriestley closed T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException as Resolved.

Sounds good, let us know if you run into anything else.

Mar 25 2016, 2:43 PM · Passphrase, Bug Report
• bitglue added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

Latest version seems to have fixed this. Sorry for the noise, and thanks for building such an awesome software.

Mar 25 2016, 2:41 PM · Passphrase, Bug Report
epriestley added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

For completeness, T10651 is the earlier report.

Mar 25 2016, 2:36 PM · Passphrase, Bug Report
epriestley added a comment to T10665: Incorrect divergent heads message on mercurial push.

I think this should now be resolved in HEAD of master. It will likely promote to stable in about 24 hours.

Mar 25 2016, 2:35 PM · Bug Report
epriestley closed T10665: Incorrect divergent heads message on mercurial push as Resolved by committing Restricted Diffusion Commit.
Mar 25 2016, 2:34 PM · Bug Report
chad added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

(fantastic bug reporting, otherwise)

Mar 25 2016, 2:33 PM · Passphrase, Bug Report
epriestley added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

I think that stuff isn't linking because someone is pushing their own branches to the upstream staging repository, making the commits ambiguous. That is, e3f89279f912 is ambiguous as either rPe3f89279f912 or rSTAGINGe3f89279f912.

Mar 25 2016, 2:32 PM · Passphrase, Bug Report
chad added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

rPe3f89279f912 should have resolved this, I believe.

Mar 25 2016, 2:28 PM · Passphrase, Bug Report
• bitglue added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

Doh. Give me a minute, testing with 3493d9d5138e0a630624bced548090163ce9be8a now.

Mar 25 2016, 2:27 PM · Passphrase, Bug Report
epriestley added a revision to T10665: Incorrect divergent heads message on mercurial push: Restricted Differential Revision.
Mar 25 2016, 2:26 PM · Bug Report
chad added a comment to T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.

Your install is out of date.

Mar 25 2016, 2:23 PM · Passphrase, Bug Report
epriestley added a comment to T10665: Incorrect divergent heads message on mercurial push.

Ah, thanks! To hit this, I think we need to push a mixture of changes:

Mar 25 2016, 2:22 PM · Bug Report
• bitglue created T10666: Unable to add API token credential for CircleCI: PhabricatorDataNotAttachedException.
Mar 25 2016, 2:16 PM · Passphrase, Bug Report
fedegiova added a comment to T10665: Incorrect divergent heads message on mercurial push.

I've cloned the repo to a test machine, so I'm available to make more tests if you need

Mar 25 2016, 2:11 PM · Bug Report
fedegiova added a comment to T10665: Incorrect divergent heads message on mercurial push.

I've rolled back my state to better identify the steps.

Mar 25 2016, 2:10 PM · Bug Report
epriestley added a comment to T10665: Incorrect divergent heads message on mercurial push.

The only way I can see to get to an error state is if both branches had the same head before the push? I'm not sure how to build that state, although maybe I'm just missing some subtlety of hg branch or some sequence of merge/close operations which can force it.

Mar 25 2016, 12:03 PM · Bug Report
epriestley added a comment to T10665: Incorrect divergent heads message on mercurial push.

Using mercurial push a set of commits that updates multiple branches at the same time

Mar 25 2016, 11:56 AM · Bug Report
fedegiova created T10665: Incorrect divergent heads message on mercurial push.
Mar 25 2016, 10:39 AM · Bug Report

Mar 24 2016

sascha-egerer added a comment to T10661: Can't create milestone.

Ok i found the reason for the problem.
If you set Applications → Projects → Edit Policies → Default View Policy to Project Memebers i does not work. You can still create normal projects and subprojects but you can't create milestones. I think that is because milestones do not have members but the permissions are evaluated against the "project members".

Mar 24 2016, 9:52 PM · Projects (v3), Bug Report
chad added a comment to T10663: macro thumbnail code sometimes fails.

This isn't a valid bug report. Please see Contributing Bug Reports and Providing Reproduction Steps.

Mar 24 2016, 6:45 PM · Macros, Bug Report
chad closed T10663: macro thumbnail code sometimes fails as Invalid.
Mar 24 2016, 6:43 PM · Macros, Bug Report
eadler created T10663: macro thumbnail code sometimes fails.
Mar 24 2016, 6:09 PM · Macros, Bug Report
brice added a comment to T10662: Cycle redirect at first connection.

Thanks for your message. Yes it was a server issue with Apache 2.4. Acces was denied. It's ok now.

Mar 24 2016, 4:35 PM · Bug Report
chad added a comment to T10662: Cycle redirect at first connection.

This error for a new installation is usually do to a mis-configured webserver redirect. If you are using Apache, make sure to place the rules in httpd.conf or similar, not .htaccess.

Mar 24 2016, 4:28 PM · Bug Report
epriestley closed T10662: Cycle redirect at first connection as Invalid.

This report does not include valid reproduction instructions. These instructions are required; see Providing Reproduction Steps for help.

Mar 24 2016, 4:17 PM · Bug Report
brice created T10662: Cycle redirect at first connection.
Mar 24 2016, 4:12 PM · Bug Report
sascha-egerer added a comment to T10661: Can't create milestone.

Ok, then I'll investigate further and give feedback as soon as i've found something

Mar 24 2016, 2:30 PM · Projects (v3), Bug Report
chad added a comment to T10661: Can't create milestone.

No luck, I tried the same settings you just posted, an admin as edit policy, also in the group view and join policy. I couldn't reproduce your screenshots.

Mar 24 2016, 2:17 PM · Projects (v3), Bug Report
sascha-egerer added a comment to T10661: Can't create milestone.

Yes I'm member of all those groups as described above. I'm also an "Admin"

Mar 24 2016, 2:13 PM · Projects (v3), Bug Report
chad added a comment to T10661: Can't create milestone.

I don't know the relation of your account to those policies. Are you a member of those groups?

Mar 24 2016, 2:09 PM · Projects (v3), Bug Report
sascha-egerer added a comment to T10661: Can't create milestone.

I've posted a screenshot of my settings or do you need anything else?

Mar 24 2016, 2:08 PM · Projects (v3), Bug Report
chad added a comment to T10661: Can't create milestone.

Generally easier if you just tell us what your settings are that reproduce the issue, rather than us try to reverse engineer all the choices.

Mar 24 2016, 2:07 PM · Projects (v3), Bug Report
sascha-egerer added a comment to T10661: Can't create milestone.

Yeah i did basically the same. What are the permissions of your project?

Mar 24 2016, 1:36 PM · Projects (v3), Bug Report
epriestley added a comment to T10661: Can't create milestone.

I can't reproduce this. Here's what I did:

Mar 24 2016, 11:45 AM · Projects (v3), Bug Report
sascha-egerer created T10661: Can't create milestone.
Mar 24 2016, 11:39 AM · Projects (v3), Bug Report
kornrunner added a comment to T10660: Php 7 installation.

PHP7 is using APCu. For those in need of APC (backwards compatibility) stuff - there's https://pecl.php.net/package/apcu_bc / https://github.com/krakjoe/apcu-bc

Mar 24 2016, 10:09 AM · Bug Report
avivey merged task T10660: Php 7 installation into T9640: Make Phabricator compatible with PHP7.
Mar 24 2016, 7:01 AM · Bug Report
avivey added a comment to T10660: Php 7 installation.

ATM, PHP 7 is not officially supported; See T9640. Your best route forward is to use PHP 5.

Mar 24 2016, 7:01 AM · Bug Report
svmagnum updated the task description for T10660: Php 7 installation.
Mar 24 2016, 5:44 AM · Bug Report