Markers (TODO / FIXME)
data/SugarBean.php5
| Type | Line | Description |
|---|---|---|
| todo | 175 | This should be replaced by altering the current user before the call to save. |
| todo | 1651 | remove this mechanism and replace with mechanism exclusively based on the vardefs |
| fixme | 3417 | Bug? we should remove the magic number -99 |
| todo | 4762 | this function needs adjustment |
| todo | 1259 | Add support for field type validation and encoding of parameters. |
soap/SoapRelationshipHelper.php1
| Type | Line | Description |
|---|---|---|
| fixme | 221 | This is not a DB-safe code. Does not work on SQL Server & Oracle. |
service/v4/SugarWebServiceUtilv4.php1
| Type | Line | Description |
|---|---|---|
| todo | 615 | handle role |
modules/ModuleBuilder/views/view.labels.php1
| Type | Line | Description |
|---|---|---|
| todo | 61 | Bundle Studio and ModuleBuilder label handling to increase maintainability. |
modules/ModuleBuilder/views/view.tree.php1
| Type | Line | Description |
|---|---|---|
| todo | 59 | Bundle Studio and ModuleBuilder label handling to increase maintainability. |
modules/ModuleBuilder/controller.php1
| Type | Line | Description |
|---|---|---|
| todo | 544 | FOR MB |
modules/ModuleBuilder/MB/MBModule.php1
| Type | Line | Description |
|---|---|---|
| todo | 815 | replace by similar mechanism to StudioModule to determine the list of available views for this module |
modules/ModuleBuilder/parsers/views/UndeployedSubpanelImplementation.php1
| Type | Line | Description |
|---|---|---|
| todo | 66 | history |
modules/ModuleBuilder/parsers/views/SubpanelMetaDataParser.php1
| Type | Line | Description |
|---|---|---|
| todo | 63 | check the implementations |
modules/ModuleBuilder/parsers/relationships/AbstractRelationships.php1
| Type | Line | Description |
|---|---|---|
| todo | 178 | error handling in case we get a badly formed definition and hence relationship |
modules/OptimisticLock/LockResolve.php1
| Type | Line | Description |
|---|---|---|
| fixme | 48 | Replace the comparison here with a function from SugarWidgets |
modules/SchedulersJobs/SchedulersJob.php4
| Type | Line | Description |
|---|---|---|
| todo | 153 | figure out what error is thrown when no more apache instances can be spun off |
| todo | 176 | make the below configurable |
| todo | 253 | what we do if job fails, notify somebody? |
| todo | 262 | what we do if job fails, notify somebody? |
modules/Import/sources/ImportFile.php1
| Type | Line | Description |
|---|---|---|
| todo | 259 | Add auto detection for field delim and qualifier properteis. |
modules/Users/views/view.edit.php1
| Type | Line | Description |
|---|---|---|
| fixme | 136 | Translate error prefix |
modules/DynamicFields/DynamicField.php1
| Type | Line | Description |
|---|---|---|
| todo | 495 | fix this - saveToVardefs() updates GLOBAL['dictionary'] correctly, obtaining its information directly from the fields_meta_data table via buildCache()... |
modules/Schedulers/SchedulerDaemon.php2
| Type | Line | Description |
|---|---|---|
| todo | 450 | figure if this is better than the above check |
| todo | 526 | make sure this will fail gracefully |
modules/Meetings/Meeting.php1
| Type | Line | Description |
|---|---|---|
| todo | 813 | do we really need focus, name and view params for this function |
modules/History/metadata/subpaneldefs.php1
| Type | Line | Description |
|---|---|---|
| todo | 51 | try and merge with the activities |
modules/Notes/Note.php1
| Type | Line | Description |
|---|---|---|
| todo | 230 | Seems odd we need to clear out these values so that list views don't show the previous rows value if current value is blank |
modules/Activities/metadata/subpaneldefs.php2
| Type | Line | Description |
|---|---|---|
| todo | 54 | remove name=nothing and make it safe |
| todo | 55 | update layout editor to match new file structure |
modules/Campaigns/QueueCampaign.php1
| Type | Line | Description |
|---|---|---|
| todo | 161 | SM: may want to move this to query clause above instead |
modules/SugarFeed/Dashlets/SugarFeedDashlet/SugarFeedDashlet.php1
| Type | Line | Description |
|---|---|---|
| fixme | 264 | Actually calculate the oldest sugar feed we can see, once we get an API that supports this sort of filter. |
modules/Currencies/Currency.php1
| Type | Line | Description |
|---|---|---|
| todo | 482 | refactor this. unit tests should not have static dependencies |
modules/Calendar/CalendarUtils.php2
| Type | Line | Description |
|---|---|---|
| todo | 187 | CHECK DATETIME VARIABLE |
| todo | 368 | CHECK DATETIME VARIABLE |
modules/Configurator/metadata/SugarpdfSettingsdefs.php1
| Type | Line | Description |
|---|---|---|
| todo | 263 | translate |
modules/Leads/views/view.convertlead.php1
| Type | Line | Description |
|---|---|---|
| todo | 246 | This should probably be on the smarty template |
modules/Contacts/Popup_picker.php1
| Type | Line | Description |
|---|---|---|
| todo | 114 | cleanup the construction of $addform |
modules/Administration/controller.php1
| Type | Line | Description |
|---|---|---|
| todo | 77 | find way to enforce order |
modules/vCals/HTTP_WebDAV_Server_vCal.php4
| Type | Line | Description |
|---|---|---|
| todo | 113 | throw if not a directory |
| todo | 286 | support this if ext/zlib filters are available |
| todo | 306 | we have to ensure that implementations support this or send 501 instead |
| todo | 312 | maybe we can just pretend here? |
modules/Groups/EditView.php1
| Type | Line | Description |
|---|---|---|
| todo | 53 | figure out why i have to hard-code this data load? |
modules/Emails/Email.php1
| Type | Line | Description |
|---|---|---|
| todo | 1139 | make this update? |
modules/InboundEmail/InboundEmail.php3
| Type | Line | Description |
|---|---|---|
| todo | 4347 | add team security to this query |
| todo | 4537 | figure out if the since date is UDT |
| todo | 5126 | - set team_id, team_set for new UI |
install/install_utils.php1
| Type | Line | Description |
|---|---|---|
| fixme | 1769 | where should it be? |
include/SugarDateTime.php3
| Type | Line | Description |
|---|---|---|
| todo | 152 | better way to not risk locale stuff problems? |
| fixme | 576 | locale? |
| fixme | 585 | locale? |
include/ListView/ListViewSmarty.php1
| Type | Line | Description |
|---|---|---|
| todo | 221 | Cleanup, better logic for which modules are exempt from the new messaging. |
include/ListView/ListView.php1
| Type | Line | Description |
|---|---|---|
| todo | 290 | expensive and needs to be removed and done better elsewhere |
include/ListView/ListViewXTPL.php1
| Type | Line | Description |
|---|---|---|
| todo | 142 | make popups / listview check saving the same |
include/SugarOAuthServer.php1
| Type | Line | Description |
|---|---|---|
| fixme | 97 | add ts/nonce verification |
include/MassUpdate.php2
| Type | Line | Description |
|---|---|---|
| todo | 213 | define filter array here to optimize the query |
| todo | 490 | allow ACL access for Delete to be set false always for users |
include/SugarQueue/SugarJobQueue.php3
| Type | Line | Description |
|---|---|---|
| todo | 171 | soft-delete old done jobs? |
| todo | 194 | tranaction start? |
| todo | 218 | commit/check? |
include/Popups/PopupSmarty.php1
| Type | Line | Description |
|---|---|---|
| todo | 531 | cleanup the construction of $addform |
include/Popups/Popup_picker.php1
| Type | Line | Description |
|---|---|---|
| todo | 133 | cleanup the construction of $addform |
include/upload_file.php2
| Type | Line | Description |
|---|---|---|
| fixme | 411 | Translate |
| fixme | 423 | Translate |
include/externalAPI/cmis_repository_wrapper.php15
| Type | Line | Description |
|---|---|---|
| todo | 120 | Make this work with cookies |
| todo | 215 | Make this storage optional |
| todo | 304 | Review |
| todo | 386 | Maybe use ->length=0 to even detect null values |
| todo | 402 | RRM FIX THIS |
| todo | 464 | RRM FIX THIS |
| todo | 693 | Fix Type Caching with missing properties |
| todo | 760 | Refactor Type Entries Caching |
| todo | 778 | Refactor Type Entries Caching |
| todo | 782 | Need GenURLQueryString Utility |
| todo | 784 | Need right URL |
| todo | 840 | Need GenURLQueryString Utility |
| todo | 850 | Need GenURLQueryString Utility |
| todo | 860 | Need GenURLQueryString Utility |
| todo | 1112 | Need Proper Query String Handling |
include/externalAPI/Base/ExternalAPIBase.php1
| Type | Line | Description |
|---|---|---|
| fixme | 70 | check if the bean is validated, if not, refuse it? |
include/externalAPI/Base/OAuthPluginBase.php2
| Type | Line | Description |
|---|---|---|
| fixme | 188 | error checking here |
| fixme | 200 | error checking here |
include/Dashlets/DashletRssFeedTitle.php1
| Type | Line | Description |
|---|---|---|
| todo | 68 | use curl with waiting timeout instead of fopen |
include/TimeDate.php5
| Type | Line | Description |
|---|---|---|
| fixme | 1102 | if the string is short, should we assume date or time? |
| todo | 1172 | should we return clone? |
| todo | 315 | add caching |
| todo | 345 | add caching |
| todo | 1909 | There is hardcoded HTML in here that does not allow for localization of the AM/PM am/pm Strings in this drop down menu. Also, perhaps change to the substr_count function calls to strpos TODO: Remove after full switch to fields |
include/Sugarpdf/Sugarpdf.php2
| Type | Line | Description |
|---|---|---|
| todo | 424 | ISSUE - width in % for the td have to be multiply by the number of column. |
| todo | 630 | explode with space is not the best things to do... |
include/generic/SugarWidgets/SugarWidgetSubPanelTopCreateTaskButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 104 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidgetSubPanelTopButtonQuickCreate.php1
| Type | Line | Description |
|---|---|---|
| todo | 114 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidgetSubPanelTopScheduleMeetingButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 104 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidgetSubPanelTopButton.php2
| Type | Line | Description |
|---|---|---|
| todo | 52 | rename defines to layout defs and make it a member variable instead of passing it multiple layers with extra copying. |
| todo | 190 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidgetSubPanelRelFieldEditButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 42 | Rename this to edit link |
include/generic/SugarWidgets/SugarWidgetSubPanelTopScheduleCallButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 104 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidgetSubPanelEditButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 42 | Rename this to edit link |
include/generic/SugarWidgets/SugarWidgetSubPanelTopCreateNoteButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 104 | move this out and get $additionalFormFields working properly |
include/generic/SugarWidgets/SugarWidget.php1
| Type | Line | Description |
|---|---|---|
| todo | 39 | move me out of generic |
include/generic/SugarWidgets/SugarWidgetFielddatetimecombo.php1
| Type | Line | Description |
|---|---|---|
| todo | 49 | now for date time field , we just search from date start to date end. The time is from 00:00:00 to 23:59:59 |
include/generic/SugarWidgets/SugarWidgetSubPanelCloseButton.php1
| Type | Line | Description |
|---|---|---|
| todo | 42 | Rename this to close button field |
include/clean.php1
| Type | Line | Description |
|---|---|---|
| todo | 116 | remove this later! |
include/SubPanel/SubPanel.php1
| Type | Line | Description |
|---|---|---|
| todo | 166 | what about unions? |
include/SubPanel/registered_layout_defs.php1
| Type | Line | Description |
|---|---|---|
| todo | 48 | add global memory cache support here. If there is an in memory cache, leverage it. |
include/controller/Controller.php2
| Type | Line | Description |
|---|---|---|
| todo | 262 | jgreen - this is only single axis check_wall mechanism, will need to upgrade this to double axis |
| todo | 320 | jgreen - Add dual axis motion |
include/HTMLPurifier/HTMLPurifier.standalone.php36
| Type | Line | Description |
|---|---|---|
| fixme | 611 | ^^ not always, see start and value of list items |
| todo | 12659 | create custom child-definition for noscript that |
| todo | 12665 | convert this to new syntax, main problem is getting |
| todo | 60 | We need an easier way to inject strategies using the configuration object. |
| todo | 1251 | Refactor duplicate elements into common class (probably using composition, not inheritance). |
| todo | 1341 | Reconsider some of the public member variables |
| todo | 2216 | Unit test |
| todo | 2455 | Create a separate maintenance file advanced users can use to cache their custom HTMLDefinition, which can be loaded via a configuration directive |
| todo | 2455 | Implement memcached |
| todo | 3953 | Refactor interface so that configuration/context is determined upon instantiation, no need for messy generateFromTokens() calls |
| todo | 3953 | Make some of the more internal functions protected, and have unit tests work around that |
| todo | 4185 | This really ought to be protected, but until we have a facility for properly generating HTML here w/o using tokens, it stays public. |
| todo | 4588 | Give this its own class, probably static interface |
| todo | 4634 | Consider making some member functions protected |
| todo | 5337 | Allow injectors to request a re-run on their output. This would help if an operation is recursive. |
| todo | 5611 | Make it private, fix usage in HTMLPurifier_LanguageTest |
| todo | 5683 | Implement conditionals? Right now, some messages make reference to line numbers, but those aren't always available |
| todo | 5741 | Serialized cache for languages |
| todo | 6311 | Consider making protected |
| todo | 6363 | Consider making protected |
| todo | 6875 | Port DirectLex to use this |
| todo | 8065 | Make protected |
| todo | 8847 | Make protected |
| todo | 9488 | Make protected |
| todo | 9494 | Make protected |
| todo | 11653 | Make protected |
| todo | 11663 | Make protected |
| todo | 11673 | Make protected |
| todo | 12881 | Figure out how to protect some of these methods/properties |
| todo | 12910 | Wildcard matching and error reporting when an added or subtracted fix has no effect. |
| todo | 13366 | Ensure all states are unit tested, including variations as well. |
| todo | 13366 | Make a graph of the flow control for this Injector. |
| todo | 14281 | Reread XML spec and document differences. |
| todo | 14817 | Enable nodes to be bubbled out of the structure. |
| todo | 16639 | Validate the email address |
| todo | 16639 | Filter allowed query parameters |
include/HTMLPurifier/standalone/HTMLPurifier/Filter/ExtractStyleBlocks.php1
| Type | Line | Description |
|---|---|---|
| todo | 36 | Extend to indicate non-text/css style blocks |
include/HTMLPurifier/standalone/HTMLPurifier/Lexer/PEARSax3.php1
| Type | Line | Description |
|---|---|---|
| todo | 3 | Generalize so that XML_HTMLSax is also supported. |
include/HTMLPurifier/standalone/HTMLPurifier/Printer/ConfigForm.php1
| Type | Line | Description |
|---|---|---|
| todo | 3 | Rewrite to use Interchange objects |
include/HTMLPurifier/standalone/HTMLPurifier/Printer/HTMLDefinition.php1
| Type | Line | Description |
|---|---|---|
| todo | 231 | Also add information about internal state |
include/utils.php3
| Type | Line | Description |
|---|---|---|
| todo | 598 | Update to use global cache |
| todo | 1589 | Clint 5/20 - Make this function more generic so that it can take in the target form and field names as variables |
| todo | 4115 | this won't work completely right until we impliment css compression and combination for now, we'll just include the last css file found. |
include/database/MssqlManager.php2
| Type | Line | Description |
|---|---|---|
| fixme | 573 | this looks really bad. Probably source for tons of bug |
| fixme | 1988 | go back to the original DB |
include/database/DBManager.php2
| Type | Line | Description |
|---|---|---|
| todo | 659 | : refactor engine param to be more generic |
| todo | 745 | : refactor engine param to be more generic |
include/SugarFields/Fields/Collection/ViewSugarFieldCollection.php1
| Type | Line | Description |
|---|---|---|
| todo | 473 | the 'select' button is not fully working. We should use the sqs_objects in open_popup instead of the parameter. |
include/MVC/View/views/view.gs.php1
| Type | Line | Description |
|---|---|---|
| todo | 158 | Handle enums since we are destroying the key. |
include/connectors/utils/ConnectorUtils.php1
| Type | Line | Description |
|---|---|---|
| todo | 92 | if we don't know this connector, should we decide it's enabled or disabled? |
include/utils/layout_utils.php2
| Type | Line | Description |
|---|---|---|
| todo | 305 | refactor this into the base Popup_Picker class |
| todo | 351 | refactor this into the base Popup_Picker class |
include/utils/db_utils.php1
| Type | Line | Description |
|---|---|---|
| todo | 73 | Make this utilize the external caching mechanism after re-testing (see log on r25320). Bug 49489 - removed caching of to_html strings as it was consuming memory and never releasing it |
include/utils/LogicHook.php1
| Type | Line | Description |
|---|---|---|
| todo | 229 | Make a factory for these classes. Cache instances accross uses |
include/SugarCache/SugarCache.php1
| Type | Line | Description |
|---|---|---|
| todo | 100 | make it so developers can extend this somehow |