Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NFR] Voting for your favorite NFR #13855

Open
niden opened this Issue Feb 23, 2019 · 28 comments

Comments

Projects
None yet
@niden
Copy link
Member

niden commented Feb 23, 2019

There are a lot of great ideas regarding the framework and additional functionality to help developers. Sadly we cannot do everything in one go, so we have been concentrating in bugs mostly and neglecting NFRs. As such, there have been pages and pages of issues as NFRs that never got any attention.

In an effort to introduce functionality that the community wants mostly, please read the list below (link to the original issue for reference) and vote in the comments by using the number next to each line below. At the end we will gather the top 5-10 (depending on time constraints) and concentrate on those NFRs.

This issue will be kept open to track the list of NFRs so that we can address them all if possible. If a NFR does not get any support, it does not mean that we will ignore it.

Thank you

Application

  • 3 - Application: When using controller as micro app handler, Controller::initialize should be invoked #1036

Assets

  • 4 - Assets: Assets\Collection: Helpers for combining JS/CSS #2009
  • 5 - Assets: Phalcon\Assets - Subresource Integrity #13333

Cache

  • 6 - Cache: Cache Compression #13740
  • 7 - Cache: Cache Sub-Folders #13797
  • 8 - Cache: Libmemcached do not setup setSaslAuthData on Memcached #13394

Compilation

  • 9 - Compilation: Add generated files /ext to .gitignore #12284
  • 10 - Compilation: Consider purging ancient ext/ directory data #13727

DI

  • 12 - DI: Make shared instances removable #13440
  • 13 - DI: Phalcon\Di add alias #13042

Dispatcher

  • 16 - Dispatcher: beforeThrowable, beforeError in dispatcher #12289
  • 17 - Dispatcher: setDefaultNamespace() Alias setNamespace() #13814

Forms

  • 20 - Forms: Forms should also have attributes #13646
  • 21 - Forms: Support of nested forms and adding collections of elements to the form #11579

Http

  • 22 - Http: HTTP/2 Preload support #13128
  • 23 - Http: SetLocale() not support multiple parameters #12202
  • 25 - Http: generate URL for route with hostName #9913

Modules

  • 26 - Modules: Add module name to module class name format in Application::registerModules #12252

Mongo

  • 27 - Mongo: Any plan about support PHP MongoDB Driver? #11156
  • 28 - Mongo: Phalcon3/PHP7 Mongo Backend Cache support #12074

ODM

  • 29 - ODM: Abstraction layer for ODM #507

ORM

  • 30 - ORM: Allow size parameter for datetime and timestamp columns #13297
  • 31 - ORM: Cascade update in ORM #13804
  • 32 - ORM: Different types of messages in virtual foreign keys #13801
  • 33 - ORM: Factory for Metadata #13780
  • 35 - ORM: MySQL point field support missing #13670
  • 36 - ORM: Phalcon DB - Change Column (Rename) #12805
  • 37 - ORM: Phalcon JSON Flat-File CRUDS Support #13783
  • 38 - ORM: Phalcon\Mvc\Model\Query\BuilderInterface Definition #13021
  • 40 - ORM: ReusableInterface #13474
  • 41 - ORM: UPDATE query with JOIN in PHQL #11637
  • 42 - ORM: Way of improving generic database setup #13647
  • 44 - ORM: getRealSQLStatement and getSQLStatement has no differences #12196

Translation

  • 50 - Translation: Support 'context' in Gettext (msgctxt) #12201

Validation

  • 52 - Validation: Change field(s) in failed message #13685
  • 53 - Validation: Validation messages #13208
  • 54 - Validation: Why not to use ioc/factory style for validators like filters #13567

View/Volt

  • 56 - View: View active render path not return to prev path after partial render #13554
  • 57 - Volt: Allow \Phalcon\Mvc\View\Engine\Volt to accept Phalcon\Mvc\View\Engine\Volt\Compiler in the constructor #2700
  • 61 - Volt: Volt with "Literal" tag #1253

Update: 2019-03-11

@niden niden self-assigned this Feb 23, 2019

This was referenced Feb 23, 2019

@Jeckerson

This comment has been minimized.

Copy link
Member

Jeckerson commented Feb 24, 2019

Really hard to select one from big list...

  • 1
  • 3
  • 7
  • 15
  • 18
  • 19
  • 20
  • 23
  • 24
  • 49
  • 55
@niden

This comment has been minimized.

Copy link
Member Author

niden commented Feb 24, 2019

@Jeckerson Yes it is a big list. However we will keep this issue open so as to get as many people to have a look and decide on what is more important to be addressed. Take your time and give us input whenever you can.

@StudioMaX

This comment has been minimized.

Copy link

StudioMaX commented Feb 24, 2019

@niden maybe you can choose by the most commented/most voted issues?

@JABirchall

This comment has been minimized.

Copy link

JABirchall commented Feb 24, 2019

I vote for

  • 22
  • 43
  • 44
  • 45
  • 47
  • 51
  • 60
@Mouvedia

This comment has been minimized.

Copy link

Mouvedia commented Feb 24, 2019

49

@scrnjakovic

This comment has been minimized.

Copy link
Contributor

scrnjakovic commented Feb 24, 2019

  • 19
  • 34
  • 39
  • 45

Are top in my book

@dschissler

This comment has been minimized.

Copy link
Contributor

dschissler commented Feb 24, 2019

I vote for my two issues which already have twice approved PRs.

@ekmst

This comment has been minimized.

Copy link
Contributor

ekmst commented Feb 24, 2019

2
25
31

@TimurFlush

This comment has been minimized.

Copy link

TimurFlush commented Feb 24, 2019

  • 1

  • 3

  • 6

  • 11

  • 16

  • 18

  • 19

  • 31

  • 33

  • 45 🔥🔥🔥

  • 46

  • 47

  • 51 🔥

  • 55

@angelvega93

This comment has been minimized.

Copy link

angelvega93 commented Feb 24, 2019

  • 6
  • 34
  • 60
@Ultimater

This comment has been minimized.

Copy link
Member

Ultimater commented Feb 24, 2019

  • 1
  • 15
  • 27
  • 39
  • 45
  • 47
@lanlin

This comment has been minimized.

Copy link

lanlin commented Feb 25, 2019

  • 15
  • 16
  • 22
  • 26
  • 27
  • 45
  • 46
@olegatro

This comment has been minimized.

Copy link

olegatro commented Feb 25, 2019

  • 1
  • 19
  • 39
  • 45
@grigoriy-ivanov

This comment has been minimized.

Copy link

grigoriy-ivanov commented Feb 25, 2019

  • 1
  • 11
  • 19
  • 21
  • 22
  • 33
  • 34
  • 38
  • 39
  • 40
  • 45
  • 48
  • 51
  • 54
  • 60
@Jurigag

This comment has been minimized.

Copy link
Member

Jurigag commented Feb 25, 2019

Well overall i don't want to vote on anything, but overall i think any improvements made to DI and PHQL/ORM will be great and we should focus on those.

@quasipickle

This comment has been minimized.

Copy link
Contributor

quasipickle commented Feb 25, 2019

  • 2 (use REQUEST_URI by default)
  • 4 (Combine assets)
  • 11 (Argon2)
  • 24 (Get Best Language )
  • 34 (Many-to-Many)
  • 38 (BuilderInterface Definition)
  • 53 (Validation messages)
  • 59 (Embedding blocks)
  • 60 (Output filters)

In general I think it would be a good practice to fix any low hanging fruit (bolded) before adding new stuff

@BasSteenbeeke

This comment has been minimized.

Copy link

BasSteenbeeke commented Feb 26, 2019

  • 18 - Forms: Add proper support to \Phalcon\Form for Radio button #13461
@mj-tm

This comment has been minimized.

Copy link

mj-tm commented Feb 26, 2019

  • 39 - ORM: Query builder for raw sql #2052
    Thank you as always
@hisune

This comment has been minimized.

Copy link

hisune commented Feb 27, 2019

  • 27
  • 28
    Waited for too many years...
@cnyyk

This comment has been minimized.

Copy link

cnyyk commented Mar 4, 2019

35 - ORM: MySQL point field support missing #13670

@woody712

This comment has been minimized.

Copy link

woody712 commented Mar 5, 2019

47

@richmilns

This comment has been minimized.

Copy link

richmilns commented Mar 5, 2019

  • 2
  • 15
  • 17
  • 18
  • 25
  • 43 :)
  • 61
@Ark4ne

This comment has been minimized.

Copy link

Ark4ne commented Mar 6, 2019

  • 2 - Application: Use REQUEST_URI by Default
  • 11 - Crypt: Implement Argon2 for password hashing
  • 12 - DI: Make shared instances removable
  • 13 - DI: Phalcon\Di add alias
  • 22 - Http: HTTP/2 Preload support
  • 34 - ORM: Many to Many behaviour
  • 36 - ORM: Phalcon DB - Change Column (Rename)
  • 48 - Router: Router enhancements and changes (caching support like Jurigag/fast-micro-router-phalcon 🚀)
  • 57 - Volt: Allow Phalcon\Mvc\View\Engine\Volt to accept Phalcon\Mvc\View\Engine\Volt\Compiler in the constructor
@krazzer

This comment has been minimized.

Copy link

krazzer commented Mar 8, 2019

  • 59
  • 41
  • 45
@emiliodeg

This comment has been minimized.

Copy link
Member

emiliodeg commented Mar 9, 2019

I've been thinking like a week in this post, and I think all NFR in Application, Cache and Redis, Compilation, Crypt, DI, Dispatcher, Http, ORM and Validation are mandatories but I know our limitations... said the adobe, I do my list

  • 4
  • 5
  • 18
  • 20 *
  • 24
  • 40 *
  • 43
  • 45
  • 46
  • 47
  • 48
  • 49 *
  • 50
  • 51
  • 53 *
  • 55
  • 59
  • 60
  • 61

* I can help coding in that points which I consider the easiest for me

@niden

This comment has been minimized.

Copy link
Member Author

niden commented Mar 11, 2019

2019-03-11 - Issues with more than 2 votes have been added to the 4.1 project.

We will consider the rest as time allows.

@diegosardina

This comment has been minimized.

Copy link

diegosardina commented Mar 16, 2019

20
21
22
61

@Rajeshr34

This comment has been minimized.

Copy link

Rajeshr34 commented Mar 17, 2019

35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.