Page MenuHomePhabricator
Differential D13899

Fix new Question in Ponder
ClosedPublic
Actions

Authored by chad on Aug 14 2015, 4:33 PM.
Tags
None
Referenced Files
F15454329: D13899.diff
Sat, Mar 29, 5:57 PM
F15427832: D13899.diff
Sun, Mar 23, 4:13 PM
F15424649: D13899.id33556.diff
Sat, Mar 22, 11:37 PM
F15386523: D13899.id33553.diff
Sat, Mar 15, 12:46 AM
F15383718: D13899.id33555.diff
Fri, Mar 14, 5:40 PM
Unknown Object (File)
Feb 21 2025, 10:18 AM
Unknown Object (File)
Feb 12 2025, 4:49 PM
Unknown Object (File)
Feb 12 2025, 3:37 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPe470b6bee2d2: (stable) Promote 2015 Week 33
rP3a6c3cc5ca9e: Fix new Question in Ponder
Summary

I derped here and broke new questions, also remove old Answer constants

Test Plan

Ask a new question, see it save

Diff Detail

Repository
rP Phabricator
Branch
remove-ponder-const
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7635
Build 8309: [Placeholder Plan] Wait for 30 Seconds
Build 8308: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33553.Aug 14 2015, 4:33 PM
chad retitled this revision from to Fix new Question in Ponder.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 14 2015, 4:33 PM
epriestley accepted this revision.Aug 14 2015, 4:35 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/ponder/controller/PonderQuestionEditController.php
70–75

Slightly cleaner is to just only do this part if !$is_new.

This revision is now accepted and ready to land.Aug 14 2015, 4:35 PM
chad updated this revision to Diff 33555.Aug 14 2015, 4:46 PM
chad edited edge metadata.
  • cleaner, fix blank details
Closed by commit rP3a6c3cc5ca9e: Fix new Question in Ponder (authored by chad, committed by chad). · Explain WhyAug 14 2015, 4:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
ponder/
constants/
19 lines
controller/
8 lines
31 lines

Diff 33555

View Options

src/applications/ponder/constants/PonderQuestionStatus.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionEditController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionViewController.php

Loading...
Phabricator · Built by Phacility